Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: set API max retry #269

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: set API max retry #269

wants to merge 1 commit into from

Conversation

Dev79844
Copy link

@Dev79844 Dev79844 commented Oct 2, 2023 •
edited

Description

This PR contains code for rate limiting the apis.

Related Issues

Feat #262

Proposed Changes

I have added rate limiting using the django rest framework.

Checklist

  • My submission is formatted according to the guidelines in the contributing guide
  • My submission has a useful description
  • Code follows the established CODE_OF_CONDUCT guidelines.
  • Code has been tested thoroughly.
  • All commit messages are descriptive and follow the established commit message format.
  • Pull request description clearly describes the changes included in the pull request.
  • I have searched the repository for any relevant issues or pull requests
  • Any category I am creating has the minimum requirement of 3 items

@gdjohn4s gdjohn4s self-requested a review October 2, 2023 08:17
@gdjohn4s gdjohn4s added enhancement New feature or request backend labels Oct 2, 2023
@gdjohn4s gdjohn4s added the help wanted Extra attention is needed label Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gdjohn4s gdjohn4s Awaiting requested review from gdjohn4s

@FlavioAdamo FlavioAdamo Awaiting requested review from FlavioAdamo

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
backend enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Dev79844 @gdjohn4s