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(server): fix queue processing #3764

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

Conversation

bertho-zero
Copy link

@bertho-zero bertho-zero commented Jan 26, 2024

Description

Overseerr currently only shows the first 10 downloads of radarr or sonarr due to the default page size.

The fix requests the size of the queue before request for the content of the queue to allow more than 10 elements to be seen.

@bertho-zero bertho-zero changed the title feat(server): fix queue processing fix(server): fix queue processing Jan 26, 2024
Copy link

stale bot commented Apr 22, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Apr 22, 2024
@bertho-zero
Copy link
Author

Don't stale !

@danshilm danshilm removed the stale label Apr 25, 2024
@danshilm danshilm self-assigned this Apr 25, 2024
@OwsleyJr
Copy link
Collaborator

If we were to implement this, we would need some way to handle the pagination if there were was a large queue.

@danshilm
Copy link
Collaborator

danshilm commented Oct 2, 2024

Agree with @OwsleyJr .
I think that a page size of 25 with a limit at 500 (or maybe 250?) should be good here?
I can definitely imagine a scenario where someone's download client bugged out/got corrupted and a user finds themselves with 100's or even 1000's of items in their Sonarr/Radarr queue.

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

Successfully merging this pull request may close these issues.

3 participants