-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
GitHub App: add docs #12114
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
base: main
Are you sure you want to change the base?
GitHub App: add docs #12114
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -205,13 +205,7 @@ Webhook activation failed. Make sure you have the necessary permissions | |
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | ||
|
||
If you find this error, | ||
make sure your user has permissions over the repository. | ||
In case of GitHub, | ||
check that you have granted access to the Read the Docs `OAuth App`_ to your organization. | ||
Comment on lines
-209
to
-210
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Don't we still need to mention this but pointing to our GitHub Application instead? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There are no webhook creation related errors when using the github app. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah, I don't refer to webhooks but to the authorization of our GitHub App. The user still have to grant access to it, right? I'm taking about this https://github.com/settings/connections/applications/Iv23liE6t9Wm8swttujH There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That authorization is done when you log in. But isn't needed as long as the app is installed in the repo. But again, users are never presented with this error when using a GH app. |
||
A similar workflow is required for other supported providers. | ||
|
||
.. _OAuth App: https://github.com/settings/applications | ||
|
||
make sure your user has admin permissions over the repository. | ||
|
||
My project isn't automatically building | ||
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would finish taking about SSH keys and submodules first and then explain the GitHub case.