Skip to content

update docs to specify Metadata access #3861

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 4 commits into from
Mar 9, 2021

Conversation

georgicodes
Copy link
Contributor

@georgicodes georgicodes commented Feb 22, 2021

This PR updates the "Creating a github app" docs. It fixes an outdated instruction when creating a new app to "Leave this page with its default values". When these instructions were written, apps were created with the metadata permission by default, however this is no longer the case (slack conversation).

Creating an app with no permissions is a little confusing for firstime app users, when you go to install the app you are then faced with this screen:
image

I think instead for the purposes of this tutorial, let's have them select metadata perms, so that the rest of the screenshots in that tutorial still make sense, eg:
image

What's being changed:

rendered

Check off the following:

@janiceilene janiceilene added the github apps Content related to GitHub Apps label Feb 23, 2021
@janiceilene
Copy link
Contributor

@georgicodes Thanks so much for opening a PR! I'll get this triaged for review ⚡

Copy link

@Alyana38 Alyana38 left a comment

Choose a reason for hiding this comment

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

Замечательная

@janiceilene janiceilene added the waiting for review Issue/PR is waiting for a writer's review label Feb 26, 2021
@mchammer01 mchammer01 self-assigned this Mar 3, 2021
@mchammer01 mchammer01 dismissed stale reviews from ghost via 2f9d8d9 March 3, 2021 17:00
@mchammer01 mchammer01 enabled auto-merge March 9, 2021 08:06
Copy link
Contributor

@mchammer01 mchammer01 left a comment

Choose a reason for hiding this comment

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

@georgicodes - thanks again for raising this PR and answering my (sometimes silly) questions ✨

@mchammer01 mchammer01 merged commit 7b0dbc7 into main Mar 9, 2021
@mchammer01 mchammer01 deleted the gk/fix_deafult_perms_line_apps branch March 9, 2021 08:07
@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2021

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
github apps Content related to GitHub Apps 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.

5 participants