Skip to content

Additional options for adding Sonarr series#1969

Merged
liiight merged 2 commits intoFlexget:developfrom
qk4l:develop
Sep 24, 2017
Merged

Additional options for adding Sonarr series#1969
liiight merged 2 commits intoFlexget:developfrom
qk4l:develop

Conversation

@qk4l
Copy link
Copy Markdown
Contributor

@qk4l qk4l commented Sep 24, 2017

Motivation for changes:

Where are no options to add series to Sonarr in unmonitored state.
Sonarr starts searching episodes right after adding.
It's not always good especially if series consist of many seasons.

Detailed changes:

New options have defaults and not affected users configs.

Config usage if relevant (new plugin or updated schema):

    list_add:
      - sonarr_list:
          base_url: '{? sonarr.url ?}'
          port: '{? sonarr.port ?}'
          api_key: '{? sonarr.api ?}'
          ignoreEpisodesWithFiles: False
          ignoreEpisodesWithoutFiles: True
          searchForMissingEpisodes: False

Log and/or tests output (preferably both):

2017-09-24 15:08 DEBUG    sonarr_list   Trakt2Sonar     adding show {u'certification': u'TV-MA', u'overview': u"A gritty chronicle of the war against Colombia's most notorious drug cartels.", u'airTime': u'03:00', u'firstAired': u'2015-08-27T21:00:00Z', u'tvRageId': 37241, u'year': 2015, u'images': [{u'coverType': u'fanart', u'url': u'http://thetvdb.com/banners/fanart/original/282670-21.jpg'}, {u'coverType': u'banner', u'url': u'http://thetvdb.com/banners/graphical/282670-g.jpg'}, {u'coverType': u'poster', u'url': u'http://thetvdb.com/banners/posters/282670-13.jpg'}], u'rootFolderPath': u'/data/TV_Shows/', u'ratings': {u'votes': 77, u'value': 8.9}, u'genres': [u'Crime', u'Drama'], u'monitored': True, u'network': u'Netflix', u'title': u'Narcos', u'remotePoster': u'http://thetvdb.com/banners/posters/282670-13.jpg', u'seasonCount': 3, u'seriesType': u'standard', u'status': u'continuing', u'added': u'0001-01-01T00:00:00Z', u'tvdbId': 282670, u'tags': [], u'imdbId': u'tt2707408', u'seasonFolder': False, u'cleanTitle': u'narcos', u'sortTitle': u'narcos', u'seasons': [{u'monitored': False, u'seasonNumber': 0}, {u'monitored': True, u'seasonNumber': 1}, {u'monitored': True, u'seasonNumber': 2}, {u'monitored': True, u'seasonNumber': 3}], u'qualityProfileId ': 1, u'useSceneNumbering': False, u'addOptions': {u'searchForMissingEpisodes': True, u'ignoreEpisodesWithFiles': False, u'ignoreEpisodesWithoutFiles': False}, u'titleSlug': u'narcos', u'qualityProfileId': 0, u'profileId': 1, u'runtime': 50, u'tvMazeId': 2705} to sonarr
2017-09-24 15:08 VERBOSE  sonarr_list   Trakt2Sonar     Successfully added show Narcos to Sonarr

To Do:

'only_monitored': {'type': 'boolean', 'default': True},
'include_data': {'type': 'boolean', 'default': False}
'include_data': {'type': 'boolean', 'default': False},
'searchForMissingEpisodes': {'type': 'boolean', 'default': True},
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I know you wanted to stay true with the original Sonarr API name, but we use camel case where relevant. Please switch to that.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Is it good names?

        'search_missing_episodes':
        'ignore_episodes_without_files'
        'ignore_episodes_with_files'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

kinda long but i dont have any better ideas

@liiight liiight merged commit fa64bd4 into Flexget:develop Sep 24, 2017
@liiight
Copy link
Copy Markdown
Member

liiight commented Sep 24, 2017

thanks! care to update the wiki?

@qk4l
Copy link
Copy Markdown
Contributor Author

qk4l commented Sep 24, 2017

Sure. I have updated wiki.

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.

2 participants