addImportExclusion for Sonarr

[marc0janssen]

Hi Vivien....

It is not working for the Sonarr API.... I checked the webz and there need to be a addImportExclusion: bool too for SONARR!

Can you please verify???

[vche]

I've checked yesterday and didn't find the option sonarr to exclude the file [in the api(https://github.com/Sonarr/Sonarr/wiki/Series). Did you find documentation that this can be done using the api ?

[marc0janssen]

delete_series(series_id: Optional[int] = None, tvdb_id: Optional[int] = None, addImportExclusion: bool = False, deleteFiles: bool = False)→ arrapi.objs.reload.Series[source] Gets a Series by one of the IDs and deletes it from Sonarr.

Parameters series_id (Optional[int]) – Search by Sonarr Series ID. tvdb_id (Optional[int]) – Search by TVDb ID. addImportExclusion (bool) – Add Import Exclusion for this Series. deleteFiles (bool) – Delete Files for this Series. Returns Series for the ID given.

Return type Series

Raises ValueError – When no ID is given. NotFound – When there’s no series with that ID or when the Series hasn’t been added to Sonarr.

[marc0janssen]

https://arrapi.readthedocs.io/en/latest/sonarr.html

[vche]

Did you test this ?

That's pycliarr method. Under the hood I'm calling an api of sonarr. And while I did in sonarr similar to radarr, adding an "AdddExclusion" parameter to the request, sonarr does not document it.

However, checking sonarr soruce code, they seem to have implemented this in their v3 api they just didn't documented it.

I'll work on an update to fully move to api v3 and support this, but i'll need to do a complete retest of every command, this may take some time

[marc0janssen]

No I haven't tested this... sorry.

For me there is NO(!!) rush Vivien... I appreciate it very much, the work you do, I would love it to have but no(!) deadline...

Thank you!

[vche]

It was actually not as long as i thought. Implemented in release .16 Also added notification, blocklist, and tag apis. So that we can create,list, and remove tags. Also provide list of series/movies for a specific tag

[marc0janssen]

Yo, that's quick!! Thanks!! I will check the tags out too!!!! Thanks a lot Vivien!!!!

[vche]

reopening to confirm if the api is implemented in radarr

[marc0janssen]

Sorry Vivien, I am not sure where to reopen a ticket... I will check this out..

[vche]

display the closed tickets and comment using the « comment and re open’. I already reopened it for you. i will check if radarr implements it but i can do much it it doesn’t.

[marc0janssen]

Thanks this helps…. Sorry for the inconvenience On 26 Nov 2021, 12:21 +0100, Vivien Chene @.***>, wrote:

display the closed tickets and comment using the « comment and re open’. I already reopened it for you. i will check if radarr implements it but i can do much it it doesn’t. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[vche]

I've captured the gui traffic to reverse engineer the requests used by the front end and they're using a different name "addImportListExclusion".

I've also got the undocumented inclusion apis, which are different as well between sonarr and radarr. As this is quick i will fix this add apis for this the inclusion list to allow listing and removing the exclusions later today.

[marc0janssen]

Great work! Thanks again! On 26 Nov 2021, 13:01 +0100, Vivien Chene @.***>, wrote:

I've captured the gui traffic to reverse engineer the requests used by the front end and they're using a different name "addImportListExclusion". I've also got the undocumented inclusion apis, which are different as well between sonarr and radarr. As this is quick i will fix this add apis for this the inclusion list to allow listing and removing the exclusions later today. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[vche]

Just missing some unit tests, i will add them next week. Re opening the ticket just so that you can confirm everything works an re close it yourself after testing.

[marc0janssen]

Tested this to ... TV shows show up in in the exclusions list in Sonarr.... Merci for this too, code-wizard!

[vche]

:) de rien / you're welcome.

please close this issue once you have successfully tested everything on your end (no rush, take your time).

[marc0janssen]

Yep all is fine.... In v3 the queue json-response is different so I need some rework in the code!

The API is fine, merci!