-
-
Notifications
You must be signed in to change notification settings - Fork 145
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 setting to enable rewatching in the "Next Up" section #1792
Conversation
This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity. |
Should this pull request be converted to a draft until it can get reviewed so that it doesn't get closed? |
This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity. |
This pull request has been closed because it has been inactive for 28 days. You may submit a new pull request if desired. |
Should I submit a new pull request? |
Sorry this one slipped through the cracks. No need to remake |
No problem. Thanks! |
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.
This looks great.
Would you mind changing the user setting so that it uses the web client setting by default? The goal is to have all user settings use web client by default with options to override client side.
How should this be done? I'm currently not at my PC, but I would like to get an idea of how to implement your requested changes. |
First step is figuring out how to access web client setting. I don't know for sure but I think the web client setting for rewatched should already be saved to the global session in either Then you'd need to modify the user setting to a radio selector instead of a bool. That way you can add a third option to the setting - Use web client setting(default), enabled, and disabled. Then you would have to tweak your code to use the web client setting from Does that make sense? |
Okay, so I searched in both |
I forgot the display settings come from a different API endpoint and aren't saved to the session var https://github.com/jellyfin/jellyfin-roku/blob/master/source/utils/session.bs#L232 but there still isn't a setting for rewatched so it looks like we don't have access to the web client setting :( |
Co-authored-by: Charles Ewert <[email protected]>
Sorry I forgot about the translations. |
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.
Thanks for the PR!
Changes
Add setting to User Interface->General to enable the rewatching of episodes in the "Next Up" section.
Add translation strings for setting.
Issues
Fixes #1778