Skip to content

feat: Add option to use Seerr default quality profiles - #36

Open
raccommode wants to merge 1 commit into
varunaditya-plus:mainfrom
raccommode:feat/optional-quality-selection
Open

feat: Add option to use Seerr default quality profiles#36
raccommode wants to merge 1 commit into
varunaditya-plus:mainfrom
raccommode:feat/optional-quality-selection

Conversation

@raccommode

Copy link
Copy Markdown

Summary

  • Add an Allow users to choose quality toggle under Advanced → Request modal.
  • Keep the toggle enabled by default so existing installations retain their current behavior.
  • When disabled, movie and TV requests skip the profile picker and use the default Radarr or Sonarr profile configured in Seerr.
  • Enforce the setting in the request controller so submitted server/profile/root-folder overrides are ignored when profile selection is disabled.
  • Keep the Letterboxd bulk quality workflow independent because it has its own advanced quality settings.

How I tested

  • Ran dotnet build SeerrFin.sln -c Release with .NET 9: 0 warnings and 0 errors.
  • Ran node --check against every injected JavaScript file and the configuration page script.
  • Used a local browser fixture with the real injected modal script and verified:
    • Disabled: the modal displays Submitting request… and sends a POST containing only media type, media ID, and 4K state, without ServerId, ProfileId, or RootFolder.
    • Enabled: the existing quality picker opens and loads request-options/movie.
    • The new toggle is visible and checked by default under Advanced → Request modal.
  • Confirmed the new setting and frontend payload are embedded in the Release DLL.

Environment

  • Target: Jellyfin 10.11.x / ABI 10.11.0.0
  • Build SDK: .NET 9
  • Browser validation: local Chromium fixture on macOS

Pending real-instance validation

This PR is intentionally opened as a draft until the behavior has been verified on a real Jellyfin + Seerr installation. The server test should cover both a user with Advanced Requests permission and a standard request-only user, with the toggle enabled and disabled.

AI assistance

OpenAI Codex (GPT-5) was used to inspect the request flow, implement the focused setting, run the .NET/JavaScript checks, and exercise the local browser fixture. The change was kept separate from the localization PR to follow the repository's one-feature-per-PR rule.

@raccommode
raccommode marked this pull request as ready for review August 24, 2026 12:26
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