Skip to content

Conversation

@StarTrooper08
Copy link
Contributor

@StarTrooper08 StarTrooper08 commented Apr 19, 2021

Why:

Closes [issue link]

What's being changed:

Check off the following:

Changes made as mentioned :
#2919 (comment)

I have reviewed the code and making the PR for the same . Please have a review from your side

The changes made will closed the issue no. #2919.

What is been changed?
- The following instances:
{% data reusables.profile.access_profile %}
{% data reusables.profile.access_org %}
{% data reusables.organizations.org_settings %}

Is replaced with:
{% data reusables.profile.access_org %}
{% data reusables.organizations.org_settings %}
What  changes have been made?
Statement :
1.On the left side of your profile page, under "Organizations", click the icon for your organization.

Replaced to:
1. In the top right corner of GitHub, click your profile photo, then click "Your Organizations".
What changes have been made?
- Created a org_settings.md file and added one statement.
@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Apr 19, 2021
@janiceilene
Copy link
Contributor

👏 👏 👏 Thanks so much for finding all these changes @StarTrooper08 🎉 🎉 🎉 I'll get this up for review!

In the meantime, can you update your comment above ☝️ to use the pull request template? Having a consistent format makes it a lot easier for writers to get all the context they need when they come in to review. Thanks!

@janiceilene janiceilene added core waiting for review Issue/PR is waiting for a writer's review and removed triage Do not begin working on this issue until triaged by the team labels Apr 19, 2021
@StarTrooper08
Copy link
Contributor Author

Is the above comment is right now??

@shati-patel shati-patel linked an issue Apr 21, 2021 that may be closed by this pull request
@shati-patel shati-patel linked an issue Apr 21, 2021 that may be closed by this pull request
@ethanpalm ethanpalm self-assigned this Apr 23, 2021
@lee-dohm
Copy link
Contributor

Can you enable the checkbox to allow maintainer edits? Then I'll be able to make some quick fixes and get this merged in. Thank you! https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/allowing-changes-to-a-pull-request-branch-created-from-a-fork

@StarTrooper08
Copy link
Contributor Author

@lee-dohm pardon but I didn't get the point.

@StarTrooper08
Copy link
Contributor Author

The checkbox to allow the 'Allow edits and access to secrets by maintainers' is been selected. Should I uncheck it??

@lee-dohm
Copy link
Contributor

No, please leave it checked. I'll try pushing my changes to your PR again.

@ethanpalm ethanpalm mentioned this pull request Apr 26, 2021
6 tasks
Copy link
Contributor

@ethanpalm ethanpalm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @StarTrooper08! Thanks so much for taking the time to make all of these changes ✨

I've opened a PR to go with yours that makes a few more changes. This PR, #5732, updates images in the reusables and makes some adjustments to how the user interface elements are written. If you have any questions about it, please let me know.

Thanks again for helping to improve the GitHub docs. Please see the help wanted issues if you would like to contribute on another project.

The changes in your PR will be live on the docs.github.com site within 24 hours 😁 🎉

@ethanpalm ethanpalm merged commit 6aba61a into github:main Apr 26, 2021
@github-actions
Copy link
Contributor

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues

This was referenced Apr 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting for review Issue/PR is waiting for a writer's review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update reusable variables with new route

4 participants