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

Fix pagination on live tv pages #3409

Merged
merged 2 commits into from Feb 11, 2022
Merged

Fix pagination on live tv pages #3409

merged 2 commits into from Feb 11, 2022

Conversation

thornbill
Copy link
Member

Changes
Fixes pagination for live tv pages. (Requires unstable build of server to display the right start index.)

Issues
N/A

@thornbill thornbill added this to Active PRs in Release 10.8.0 via automation Feb 9, 2022
dmitrylyzo
dmitrylyzo approved these changes Feb 9, 2022
View reviewed changes
Copy link
Contributor

@dmitrylyzo dmitrylyzo left a comment

Choose a reason for hiding this comment

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

Can't test, but LGTM

src/controllers/list.js Show resolved Hide resolved
Release 10.8.0 automation moved this from Active PRs to Approved PRs Feb 9, 2022
@sonarcloud
Copy link

sonarcloud bot commented Feb 9, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.8% 0.8% Duplication

@thornbill thornbill merged commit 9a6ba29 into jellyfin:master Feb 11, 2022
Release 10.8.0 automation moved this from Approved PRs to Completed PRs Feb 11, 2022
@thornbill thornbill deleted the fix-tv-pagination branch February 11, 2022 15:27
@jedthe3rd jedthe3rd mentioned this pull request Mar 10, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmitrylyzo dmitrylyzo dmitrylyzo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
No open projects
Release 10.8.0
  
Completed PRs
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@thornbill @dmitrylyzo