Skip to content

feat-sync - Add customizable screensaver properties to MoonfinSettingsProfile - #272

Open
mattsigal wants to merge 1 commit into
Moonfin-Client:masterfrom
mattsigal:feature/screensaver-settings-sync
Open

feat-sync - Add customizable screensaver properties to MoonfinSettingsProfile#272
mattsigal wants to merge 1 commit into
Moonfin-Client:masterfrom
mattsigal:feature/screensaver-settings-sync

Conversation

@mattsigal

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

Adds nine new screensaver preference properties to MoonfinSettingsProfile in both Jellyfin and Emby plugin models to support cross-device synchronization of screensaver backdrops, visual components, speeds, anchor positions, sizing, and library/collection/genre filtering with Moonfin clients.

Related Issues

Link related issues or tickets separated by commas.

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • API / endpoint change
  • Settings schema change
  • Documentation update
  • Build/CI change
  • Other (describe):

Area

  • Settings sync / profiles
  • Admin defaults / config page
  • Ratings (MDBList / TMDB)
  • Notifications / Push (FCM / relay)
  • Seerr integration
  • Games / Emulators
  • Custom home rows
  • Web Client (Go to Moonfin-Core repo)
  • Other / shared

Changes Made

List the key changes included in this PR.

  • Added ScreensaverBackdrop, ScreensaverComponent, ScreensaverMovement, ScreensaverPosition, ScreensaverSize, ScreensaverContentType, ScreensaverLibraryIds, ScreensaverCollectionIds, and ScreensaverExcludedGenres to MoonfinSettingsProfile in Jellyfin/backend/Models/MoonfinSettingsProfile.cs.
  • Added matching properties with explicit [JsonPropertyName] tags to MoonfinSettingsProfile in Emby/Emby.Plugins.Moonfin/Models/MoonfinSettingsProfile.cs.

Client Impact

Does this need matching changes in a client repo (Core, Smart-TV, Roku)?

  • No client changes needed
  • Companion client PR(s) required, linked here: feat-ui - The one about Custom Screensavers Moonfin-Core#1417
  • [X ] New setting keys added. List each key and confirm it matches the client key exactly, including casing:
    • screensaverBackdrop
    • screensaverComponent
    • screensaverMovement
    • screensaverPosition
    • screensaverSize
    • screensaverContentType
    • screensaverLibraryIds
    • screensaverCollectionIds
    • screensaverExcludedGenres

Compatibility

  • Change to the settings profile is additive only, no renamed or removed properties
  • New properties use the same type the client sends (a client bool maps to bool?, an int to int?)
  • Migration added for any renamed or removed settings
  • Older clients still work, unknown fields are ignored and no keys were removed

Testing

Describe how this change was tested.

  • Built the plugin and deployed to a Jellyfin server
  • Verified against a live client (which one:)
  • Manual testing completed
  • Not tested (explain why): Tested via build tests but not deployed yet as I need to build clients to sync between.

Test Steps

  1. Built Moonfin.Server.csproj with dotnet build: Build succeeded with 0 warnings and 0 errors.
  2. Built Emby.Plugins.Moonfin.csproj with dotnet build: Build succeeded with 0 warnings and 0 errors.
  3. Verified matching key names and types against Moonfin-Core syncedFields table and unit tests.

Screenshots (if applicable)

Include config page screenshots or request/response samples where relevant.

Checklist

  • Code builds successfully
  • Code follows project style and conventions
  • No unnecessary commented-out code
  • No new warnings introduced
  • Any new setting keys match the client-side keys exactly

- Add ScreensaverBackdrop, ScreensaverComponent, ScreensaverMovement, ScreensaverPosition, ScreensaverSize, ScreensaverContentType, ScreensaverLibraryIds, ScreensaverCollectionIds, and ScreensaverExcludedGenres to MoonfinSettingsProfile in Jellyfin plugin model
- Add matching properties to MoonfinSettingsProfile in Emby plugin model
- Supports cross-device settings synchronization with Moonfin-Core screensaver enhancements
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

Build Successful

The plugin compiled successfully against .NET 8 / Jellyfin 10.10.0.

Property Value
Commit b90bfa3
Workflow Build #361

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.

1 participant