-
Notifications
You must be signed in to change notification settings - Fork 141
feat: increase rate limits #1836
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
Conversation
|
Preview for this PR was built for commit |
TC-MO
left a comment
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.
LGTM, just a few small nits:
-
we seem to mention limit explicitily in the table within API Reference (line 375
openapi.yamlmaybe we could use this opportunity to just change this into
You have exceeded the rate limit of ... requests per secondto not mention explicit number (we do it in other part of docs this way) -
in the
dataset.mdthere is also200 requests per minutementioned on the line 98, this should be 400 with this change right?
|
Thanks for the diligent review! Made the two suggested changes 👍 |
|
Preview for this PR was built for commit |
We have [doubled](https://github.com/apify/apify-core/pull/22634) API rate-limits and this PR updates the docs accordingly.
We have doubled API rate-limits and this PR updates the docs accordingly.