Skip to content
This repository has been archived by the owner on Oct 3, 2019. It is now read-only.

[ch35605] TooManyRequestsResponse 429 #16

Merged
merged 3 commits into from
Aug 15, 2019
Merged

[ch35605] TooManyRequestsResponse 429 #16

merged 3 commits into from
Aug 15, 2019

Conversation

finalprototype
Copy link
Contributor

[ch35605]

@finalprototype finalprototype requested a review from a team August 15, 2019 20:45
@mattbriancon
Copy link

Does apitools do any rate limiting?

mattbriancon
mattbriancon reviewed Aug 15, 2019
View reviewed changes
README.md Outdated
* `error_code`
* `"too_many_requests"`
```python
responses.TooManyRequestsResponse(detail="Too many request for template")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: requests

@finalprototype finalprototype merged commit e3d5635 into master Aug 15, 2019
@finalprototype finalprototype deleted the fp/res-429 branch August 16, 2019 14:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mattbriancon mattbriancon mattbriancon approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@finalprototype @mattbriancon