Skip to content

GitHub push webhooks limit with translations #12203

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

Open
nijel opened this issue May 23, 2025 · 6 comments
Open

GitHub push webhooks limit with translations #12203

nijel opened this issue May 23, 2025 · 6 comments
Labels
Needed: more information A reply from issue author is required

Comments

@nijel
Copy link
Contributor

nijel commented May 23, 2025

Details

Expected Result

I wanted to add another translation of our documentation and let RTD configure webhook at GitHub.

  • In case there is a specific error coming from GitHub, RTD should display it instead of assuming that it has to be an issue with permissions.
  • Is there some recommended approach to work around GitHub limits on number of webhooks?

Actual Result

The setup failed:

Image

I was confident that the permissions were there (it worked before), so I went to GitHub and I got a different error:

Image

So the issue is that we have just reached 20 languages and thus have 20 webhooks just from readthedocs.org.

@nijel
Copy link
Contributor Author

nijel commented May 23, 2025

I've just fixed this for this particular translation by removing webhook we no longer use. But still, we're going to hit this again once another language is translated.

@nijel nijel changed the title GitHub push webhooks limit GitHub push webhooks limit with translations May 23, 2025
@nijel
Copy link
Contributor Author

nijel commented May 23, 2025

@humitos
Copy link
Member

humitos commented May 26, 2025

We are migration our GitHub OAuth app to a GitHub App and we won't use the same webhook approach, so this may not be an issue anymore. @stsewd do you know how we are handling monorepo with multiple translations with the new GitHub App?

@humitos humitos added the Needed: more information A reply from issue author is required label May 26, 2025
@stsewd
Copy link
Member

stsewd commented May 27, 2025

This shouldn't be a problem with the new GitHub App, we basically listen to one repo, and then our backend applies that same operation to all projects linked to that repo.

@nijel
Copy link
Contributor Author

nijel commented May 27, 2025

Is there a beta where we could participate?

@stsewd
Copy link
Member

stsewd commented May 27, 2025

@nijel we are still discussing on how to expose a beta to users, you can subscribe to this issue for updates when a beta is ready #12174.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needed: more information A reply from issue author is required
Projects
None yet
Development

No branches or pull requests

3 participants