Update reusable variables with new route #2919
Comments
|
Hi, I would like to take up this issue, but I am confused on the file that will be affected (where the change will be made). |
|
@megbird Are you looking for each of the three reusables to be updated to follow the two steps you outlined?
|
|
Thank you so much! |
|
So sorry I haven't given any guidance for this! We use reusable files that we can use as variables all over the site. This issue suggests that we update three reusable and condense them down to two. For this PR, let's make the following updates:
with this
@lumenCodes I know you've already started a PR, ping me in if you have any questions! |
changes made for issue no github#2919
changes made for issue no github#2919
|
I m bit of confused for changes to be made for replacing instance .Can I get help for this?? |
This comment was marked as spam.
This comment was marked as spam.
|
@StarTrooper08 I see your two PRs and they look
and replace them with
as described in #2919 (comment). Searching in the repo on github.com will give you a bunch of partial results, so you'll want to use your editor to search instead. |
|
ok I will do it |
|
@janiceilene I have made a PR for #2919 issue . Please have a review. |
We currently use the following three steps to point a user to org settings:
{{ site.data.reusables.profile.access_profile }}data/resuables/profile{{ site.data.reusables.profile.access_org }}data/resuables/profile{{ site.data.reusables.organizations.org_settings }}data/resuables/org_settingsHowever, there is a slightly cleaner route of:
We should update the reusables to use the simpler route.
Content strategy and implementation
Please link any in-flight PRs to this issue.
The text was updated successfully, but these errors were encountered: