-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
cardigann: check for page size #14287
base: master
Are you sure you want to change the base?
Conversation
b821028
to
7de7dcd
Compare
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.
Working well!
7de7dcd
to
7e935e5
Compare
What would we use for |
This is useful for pagination rather, but not the scope of this PR. It was on mind for some time, I'll move it to cardigann pagination in the end. Actually, if the indexer isn't able to support pagination, the higher number will prevent an extra request from sonarr/radarr. |
I think some thought needs to go into the overall approach for paging before anything goes in. We've got 4 different PRs now related to Cardigann paging between the projects, so it would be good to align before moving on anything to ensure we are covering all bases. |
7e935e5
to
dc3e068
Compare
lol |
No description provided.