-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Support Sonarr v4 API #315
Comments
It looks like Radarr and Sonarr v4 both use their v3 APIs. There is nothing to do here. |
https://github.com/orgs/midarrlabs/projects/1/views/1?filterQuery=sonarr+v4 No you're right, I even have a backlog issue to add support for Sonarr V4 as you've mentioned. The API itself is still versioned v3 for both Sonarr V3 and V4 - however I have looked into it and there are slight changes. Same v3 versioned API, but it has breaking changes - episode thumbnail / screenshot schema included. Kind of mind blowing they've left the version the same. Reopening 😄 thanks again! |
https://github.com/midarrlabs/midarr-server/actions/runs/4496942700/jobs/7912075291 Just for fun, pointed CI at Sonarr V4 and it fails quite early on in the build. Looking forward to getting this building soon though. |
Sonarr v4 dropped mono and also forces auth on start, maybe one of those have something to do with it. |
I'll look into this when Sonarr V4 stable is released. Supporting a develop branch could cause all sorts of issues for us. |
Thinking I'll just pull the pin on this based on the amount of feedback I've been receiving. It will be a breaking change so |
I'll confirm here when in a build |
These versions are the current
develop
branch. It's annoying these apps how so many different channels to deal with but I've been running Sonarr and Radarr ondevelop
/ v4 for a very long time and it's been very stable. It might make sense to start expecting a v4 release of Sonarr this year sometime.The text was updated successfully, but these errors were encountered: