-
Notifications
You must be signed in to change notification settings - Fork 385
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
patch: Updated API docs regarding rate limiting #3131
base: master
Are you sure you want to change the base?
Conversation
Added a section on rate limiting to our API docs so that customers can understand why we don't publish our rate limits.
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.
Thanks for making the pull request @rhampt
You can auto accept these suggestions if you like it or make your own new changes in the same branch. In the end, I can rebase them for you to create one single commit.
Co-authored-by: Vipul Gupta <[email protected]>
Co-authored-by: Vipul Gupta <[email protected]>
Co-authored-by: Vipul Gupta <[email protected]>
Sounds good @vipulgupta2048 . I accepted and committed all suggestions. |
Added a section on rate limiting to our API docs so that customers can understand why we don't publish our rate limits.