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.

Sign up for GitHub

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

Jump to bottom

Fix json constructor #85

Merged
merged 6 commits into from
Aug 24, 2021
Merged

Fix json constructor #85

merged 6 commits into from
Aug 24, 2021

Conversation

Archmonger
Copy link
Contributor

  • Fixes bug where SonarrAPI could add series to wrong folder
  • Cleans up RadarrAPI construct_movie_json to match sonarr's equivalent
  • Adds RadarrAPI lookup_movie_by_imdb_id()

@Archmonger
Copy link
Contributor Author

Archmonger commented Aug 24, 2021 •
edited
Loading

This PR fixes #83

@Archmonger
Copy link
Contributor Author

@marksie1988 Can we get a review and merge on this ASAP? High priority issue.

@Archmonger Archmonger self-assigned this Aug 24, 2021
@Archmonger Archmonger added the P1 Urgent priority issue that should be resolved before the next release label Aug 24, 2021
@marksie1988 marksie1988 merged commit a03ba49 into totaldebug:master Aug 24, 2021
@marksie1988 marksie1988 added the type/bug Something isn't working label Aug 24, 2021
@Archmonger Archmonger deleted the series-and-movie-folder-names branch August 24, 2021 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marksie1988 marksie1988 marksie1988 approved these changes

Assignees

@Archmonger Archmonger

Labels
P1 Urgent priority issue that should be resolved before the next release type/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Archmonger @marksie1988