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

Add 'Enable Rewatching' and ' Max days' to Next Up #1212

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

JPKribs
Copy link
Contributor

@JPKribs JPKribs commented Aug 26, 2024

Resolves: #1211

Summary

This PR enables a feature found on Jellyfin-Web that limits the number of items in the 'Next Up' section. To further mirror Jellyfin-Web, this feature uses 0 as disabled (All items are shown in Next Up) and only includes the parameter for NextUpDateCutoff when this is greater than 0.

While I was in there, realized that Jellyfin-Web had an option for EnableRewatching so included that in this PR. I was basing my Parameters off of this section of Jellyfin-Web: https://github.com/jellyfin/jellyfin-web/blob/master/src/components/homesections/sections/nextUp.ts

Screenshots

iOS

New Settings (Disabled) New Settings - Disabled
New Settings (1 Day) New Settings - 1 Day
New Settings (Multiple Days) New Settings - Multiple Days

tvOS

New Settings (Location) New Settings - Location
New Settings (Disabled) New Settings - Disabled
New Settings (1 Day) New Settings - 1 Day
New Settings (Multiple Days) New Settings - Multiple Days
New Settings (Subtitle Update) New Settings - Subtitle Update

…ne view. Add parameters to the NextUpLibrary for limiting the nextup and also allowing rewatching to show up in 'UpNext'. Pull the description directly from Jellyfin-Web. Creation of CustomizationView settings to those defaults.
Joseph Kribs added 2 commits August 25, 2024 22:14
… tvOS mirror how the VideoPlayerSettings are handled where sections are indepedent views.
@sjorge
Copy link

sjorge commented Sep 9, 2024

Anything still needed for this? Would love for this to be in the next TestFlight build.

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.

Want option to limit next up in number of days like the web client
2 participants