Skip to content

fix: pass movieId to Radarr manual import - #393

Open
wylyn3d wants to merge 1 commit into
sirrobot01:betafrom
wylyn3d:fix-radarr-movieid
Open

wylyn3d wants to merge 1 commit into
sirrobot01:betafrom
wylyn3d:fix-radarr-movieid

Conversation

@wylyn3d

@wylyn3d wylyn3d commented Aug 13, 2026

Copy link
Copy Markdown

Pull Request Description

Fixes Radarr manual imports by passing the movie ID returned by Radarr's Manual Import API back to the import request.

Previously, Decypharr populated seriesId but did not read or send movieId. For Radarr imports, this resulted in ManualImport commands being submitted with movieId = 0, preventing the movie from being imported correctly.


Target Branch Check (IMPORTANT)

  • I confirm this PR is targeting the correct branch

Expected target:

  • beta (for features)

Changes Made

  • Read movie.id from the Manual Import response.
  • Add movieId to ManualImportRequestFile.
  • Pass the returned movie ID to Radarr when performing the import.
  • Keep Sonarr behavior unchanged by using omitempty.

Testing

  • Tested locally

Steps:

  1. Built Decypharr v2.5 locally with the patch applied.
  2. Re-ran the affected Radarr manual import and confirmed the request used the expected movieId instead of 0.
  3. Confirmed the download imported successfully and Radarr created the expected MovieFile.

Risks / Notes

  • The change only affects the movie ID used for Radarr manual imports.
  • movieId is omitted when empty, so Sonarr imports remain unchanged.
  • No configuration changes or migrations are required.

Screenshots (if applicable)

Not applicable; this is a backend import fix.


Checklist

  • Code builds successfully
  • No console/log errors
  • Reviewed my own code
  • Target branch is correct

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