-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Jellyfin notification API changes #5828
Comments
Are the changes this merge brings documented somewhere? I don't know what sort of time-frame we're looking at for these changes to come to stable JF, but if we want we might be able to do something akin to what we do with the qbittorrent API and support both versions. |
I can't say I know. As things stand there is simply no functional notification functionality in the stable version. Jellyfin is a little slow with releases, this is probably best left as is for now. I just happened to notice this while testing other things, and figured that this was bound to become a problem in the future. |
Also I dug around a little more, prior to the PR I mentioned apparently jellyfin/jellyfin#8952 was introduced, which cut out the part of the functionality that Sonarr actually uses. I'm not actually totally sure right now whether there are going to be alternatives given they are seemingly only talking about their own clients. |
Well, if the notifications API has been removed entirely we could just yank those parts of the code out. It doesn't look like api.jellyfin.org has been updated with these changes, since it still refers to the notification endpoints. Feel free to hit me up on Discord to discuss more back-and-forth. |
I'm guessing those are still present in Emby, so likely best kept around unchanged. The path based updates also still work(and are very useful if you have no inotify support) and are actually what got me here. |
If Jellyfin is diverging from Emby and making breaking changes then a separate connection may be warranted, though for the majority of people these notifications are probably unused and worst case they can just turn them off. |
Is there an existing issue for this?
Current Behavior
With jellyfin/jellyfin#9147 having been merged notifications will not be sent to Jellyfin properly anymore.
Expected Behavior
This should likely be updated to use the Events API.
Steps To Reproduce
Environment
What branch are you running?
Develop
Trace Logs?
Anything else?
Noticed while looking to prepare a PR for #5826.
The text was updated successfully, but these errors were encountered: