Skip to content
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

[FEATURE] Connect with other apps for notification #783

Closed
mritunjaymusale opened this issue Nov 13, 2022 · 8 comments · Fixed by #809
Closed

[FEATURE] Connect with other apps for notification #783

mritunjaymusale opened this issue Nov 13, 2022 · 8 comments · Fixed by #809
Milestone

Comments

@mritunjaymusale
Copy link

Is your feature request related to a problem? Please describe.
After downloads are finished it would be nice if I got a notification or something (when using the docker version)

Describe the solution you'd like
The *arr apps ecosystem has feature where downloads, adding of movie/show etc will automatically send a notification using their connected apps plugin. I have currently set it up in a manner where any event happens in sonarr or radarr app i get a message on my telegram bot that way it's easier to keep track of things incase something is broken or something is ready for viewing.
So I would like to a similar freature where in settings page I provide the telegram bot api key and the chat id and when there is a new subscription video downloaded and ready for viewing I will get a telegram message from my bot and I can start watching directly through your ui.
Additional context
Here's how it currently works in Sonarr :
Screenshot from 2022-11-13 14-33-16
And here's how it looks in telegram :
image

@GAS85
Copy link

GAS85 commented Nov 30, 2022

#440 is in a similar direction

@Tzahi12345 Tzahi12345 added this to the v4.3.1 milestone Jan 5, 2023
@Tzahi12345
Copy link
Owner

See latest PR #809. No promises I can get Telegram support working but I have already implemented Gotify and ntfy.

From a first look it doesn't look that hard. There's a few APIs that might be useful here:

I'm hoping to add Apprise support (as mentioned in #440), but then you'd have to host another service.

@mritunjaymusale
Copy link
Author

For telegram you don't need additional service which why I pefer that approach as compared to others that you have mentioned.
AFAIK other approaches require you to host or install some kind of additional app along with ytdl-material where as telegram-api can be bundled using the npm packages that you mentioned.

@Tzahi12345
Copy link
Owner

Tzahi12345 commented Mar 21, 2023

Added to #809! The way I have it set up now is (in the case of a download) it will send a URL of the thumbnail and then the text with more info like so:

image

Do we want to keep the thumbnail there? I'm trying to avoid adding a setting for "yes include a thumbnail" since it seems so minor.

@mritunjaymusale
Copy link
Author

I think you should keep the thumbnail on by default, since it acts as a helpful indicator when downloading a playlist or a channel.
One last thing (atleast from my side), in the message where it provides the url, is it possible to add the link to the original video aswell? For referencing sake.

@Tzahi12345
Copy link
Owner

I think you should keep the thumbnail on by default, since it acts as a helpful indicator when downloading a playlist or a channel. One last thing (atleast from my side), in the message where it provides the url, is it possible to add the link to the original video aswell? For referencing sake.

Makes sense, just added that in! 07a0ea6

@mritunjaymusale
Copy link
Author

Before this gets deployed, I came accross this another feature of telegram, where if you change the group into a supergroup you can create topics (like discord channel) so can you add another parameter which will ask for the topic / message_thread_id and the messages will land in the respective channel ?

@Tzahi12345
Copy link
Owner

It did get deployed in v4.3.1 :)

If you could create an issue for that feature request that would be helpful for tracking purposes

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 a pull request may close this issue.

3 participants