Skip to content

Correction within the About apps page #4590

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Mar 19, 2021
Merged

Correction within the About apps page #4590

merged 5 commits into from
Mar 19, 2021

Conversation

lucalves
Copy link
Contributor

Why:

I removed a link from an article that was directing itself and updated the article "Differences between apps" to "Differences between GitHub Apps and OAuth Apps.", suggestion sent by @janiceilene at #4553.

Closes #4553.

Check off the following:

@lucalves lucalves mentioned this pull request Mar 17, 2021
@lucalves
Copy link
Contributor Author

Hello, @janiceilene! I committed the suggestions. I believe it is now working as expected.

Copy link
Contributor

@janiceilene janiceilene left a comment

Choose a reason for hiding this comment

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

Thanks so much @lucalves! I'll update the branch and get this merged in once tests are passing 🍏

@janiceilene janiceilene added the ready to merge This pull request is ready to merge label Mar 19, 2021
@janiceilene janiceilene merged commit 9a48a7e into github:main Mar 19, 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge This pull request is ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Page links to itself
2 participants