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

feat(radarr): indexer #80

Merged
merged 3 commits into from
Nov 15, 2022
Merged

feat(radarr): indexer #80

merged 3 commits into from
Nov 15, 2022

Conversation

Fuochi
Copy link
Contributor

@Fuochi Fuochi commented Nov 14, 2022

  • add indexer methods
  • add indexer config methods
  • add restriction methods

Indexer is the same as sonarr. However I noticed that is a bit different in lidarr. For now, I think it'll be better to keep them separated.

Copy link
Contributor

@davidnewhall davidnewhall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me. Thanks!

@bakerboy448
Copy link

bakerboy448 commented Nov 14, 2022

Readarr will be missing the download client as Readarr/Readarr#1485 is not landed yet.

But Lidarr/Radarr/Sonarr look like their Indexer Payloads should be the same from the Indexer Schema.

Certainly suggest keeping them independent as changes may be made in one that are not immediately in another.

@Fuochi Fuochi merged commit e7470b4 into master Nov 15, 2022
@Fuochi Fuochi deleted the feature/radarr-indexer branch November 15, 2022 03:42
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.

3 participants