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.

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

fix(quota): block multi-season requests that would exceed a user's quota #1874

Merged
merged 3 commits into from Jul 13, 2021

Conversation

TheCatLady
Copy link
Collaborator

@TheCatLady TheCatLady commented Jul 7, 2021

Description

Currently, multi-season requests that would exceed a user's remaining series request quota made via the API (e.g., from Requestrr) are not being blocked if the user has at least one season request remaining.

Also add optional but missing userId to API docs for the /request POST endpoint (used for the "Request As" feature).

Screenshot (if UI-related)

N/A

To-Dos

  • Successful build yarn build

Issues Fixed or Closed

N/A

@TheCatLady TheCatLady requested a review from sct as a code owner July 7, 2021 10:38
@sct sct enabled auto-merge (squash) July 13, 2021 09:53
@sct sct merged commit 8a55f85 into develop Jul 13, 2021
@sct sct deleted the fix/series-quota branch July 13, 2021 09:58
nicospz pushed a commit to nicospz/overseerr that referenced this pull request Aug 9, 2021
…ota (sct#1874)

* fix(quota): block multi-season requests that would exceed a user's quota

* fix(docs): add missing request user option to API docs

Co-authored-by: sct <ryan@sct.dev>
@github-actions
Copy link

🎉 This PR is included in version 1.26.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants