-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Can't find resyncing instructions #5260
Comments
In reality, this is a follow-up from my comment at #4450 (comment), since I again pushed a branch to my GitHub repository and the build was not triggered, although the event was received: |
When I go to my Admin > Integrations > GitHub incoming webhook I see a button there that says "Resync webhook". Does not appear to you? Clicking it should resync the webhook and subscript to the new GH events. Also, please check that your webhooks in GH are properly configured after that at https://github.com/poliastro/poliastro/settings/hooks Example, The first one is the proper one, which is subscribed to create, delete, pull_request and push the second one is the old one. Yours, should look like the first one. Let me know! |
OK! My webhook page looks different Taking a look at the code, that's because your integration's attribute I'm not sure what that means yet I checked the Now, I need to find out how
If only |
Correct! But I guess the culprit is that my webhook is not synced. Thanks for looking into this! |
The resync option is only available if the webhook was set up using a connected GitHub account. If you disconnected your GitHub account, or reconnected, I'm not sure of the behavior. But, without this connected account, we can't attempt the API operations to resync the webhook, so it's a manual configuration piece that you need to handle on GitHub. Is you GitHub account connected to your RTD account, and does that GH user have access to the admin of the repository? |
@agjohnson I just logged out and logged in with my GH account just to try and it worked, and this user has access to the admin of the repository. |
We only show this button if the repo was imported using the Probably we can close this in favor of #3932. Or clarify the docs in the meantime. |
Do we have a proposed workaround in the meantime? I'd like to give a solution to @Juanlu001 and other people hitting this problem. |
I think there is a confusion with the feature that @Juanlu001 is mentioning
We currently listen to create/delete events to update the versions list of a project. Users still need to activate and trigger a build to new branches. If you want that feature, keep an eye in #4001 |
@stsewd oh, I think you are right. @Juanlu001 the new feature regarding how we handle webhooks on Read the Docs was about "detecting that a new branch/tag was added into the repo and create a Version for it" (not activated --which won't build by default, though). Can you confirm that the new branch created in the repo was automatically detected and added as a version on RTD? |
If I recall correctly, the new branch didn't create a new version, although it eventually did when I pushed more commits to it. If I understand correctly there's some progress in #4001 by @stsewd, so let's revisit this when I branch my project again. If you don't want to keep an issue open until July, let's close it and I will comment here if I have further problems. |
Maybe it was in a github downtime or in our servers downtime. You can revisit the webhooks page (on github and rtd) to see if the webhook was correctly handled. Let's close this for now, let us know if something doesn't work. |
(#5062 (comment))
Talking about "manually re-syncing", the webhooks docs mention
But I don't see such a thing in, for example, https://readthedocs.org/dashboard/poliastro/integrations/57915/.
The text was updated successfully, but these errors were encountered: