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

Only show PG-13 content in screensaver #3366

Merged

Conversation

nielsvanvelzen
Copy link
Member

@nielsvanvelzen nielsvanvelzen commented Feb 11, 2024

This is the first of two pull requests to add filtering based on age rating to the screensaver. This one is backportable (no string changes). The second one will add preferences and is for 0.17.

Changes

  • Add maxOfficialRating=PG-13 to library showcase in screensaver
  • Add hasParentalRating=true to library showcase in screensaver <-- this one will come with the preference in 0.17

Issues

Partially resolves #3320

@nielsvanvelzen nielsvanvelzen added the enhancement New feature or request label Feb 11, 2024
@nielsvanvelzen nielsvanvelzen added this to the v0.16.5 milestone Feb 11, 2024
@nielsvanvelzen nielsvanvelzen added the backportable Change may be backported to a point release (remove label once cherrypicked) label Feb 11, 2024
@@ -76,6 +76,9 @@
sortBy = listOf(ItemSortBy.Random),
limit = 5,
imageTypes = listOf(ImageType.BACKDROP),
// TODO: Add preferences for these two settings

Check warning

Code scanning / detekt

Flags a forbidden comment. Warning

Forbidden TODO todo marker in comment, please do the changes.
@nielsvanvelzen nielsvanvelzen merged commit 5ff02c7 into jellyfin:master Feb 11, 2024
6 checks passed
@nielsvanvelzen nielsvanvelzen deleted the screensaver-age-rating branch February 11, 2024 15:09
@nielsvanvelzen nielsvanvelzen removed the backportable Change may be backported to a point release (remove label once cherrypicked) label Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Option to exclude libraries from screensaver
2 participants