shayaantx / botdarr

Slack/Discord/Telegram/Matrix bot for accessing radarr, sonarr, and lidarr
GNU General Public License v3.0
157 stars 12 forks source link

Add support for sonarr v4 #83

Closed JulienSambre closed 1 year ago

JulienSambre commented 1 year ago

Hi!

First of all, thanks for the bot, I use it extensively with my AI to notify me when movies & co are downloading and it works like a charm.

I'd like to know if there is a plan to support Sonarr v4 API : I updated to Sonarr v4 a few weeks ago and it broke botdarr. I think there are changes on the Sonarr v4 API.

Thanks!

shayaantx commented 1 year ago

@JulienSambre I'll try to take a look at this weekend or sometime during this week, didn't know they had a new version of sonarr out. I'm also open to PR's too

JulienSambre commented 1 year ago

Thanks!

shayaantx commented 1 year ago

@JulienSambre so I been making some changes to this, not sure what version you were on prior to sonarr v4, but it looks like at least on sonarr 3.0.7.1477 at least one of the api calls is broken (one that shows downloads). I'm going thru them all again on v3 and v4 to see what is working and what isn't.

shayaantx commented 1 year ago

@JulienSambre I added this support in 5.6.0, but you need to turn on a special property to make it work (SONARR_V4=true in docker or sonarr-v4=true in properties in a manual installation - see readme)

lemme know if you have issues, cause I had to refactor a decent bit to make it easier to change/support different api versions in future (so I may have broken some stuff :) ).

shayaantx commented 1 year ago

when sonarr v3 is EOL-ed I'll remove this property fyi