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

feat: add ntfy #4703

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

feat: add ntfy #4703

wants to merge 1 commit into from

Conversation

gaetansnl
Copy link

@gaetansnl gaetansnl commented Jul 28, 2022

This PR adds support for ntfy notification server

@gaetansnl gaetansnl marked this pull request as ready for review July 29, 2022 06:04
@gaetansnl
Copy link
Author

@tidusjar I fixed the tests :)

@flowcool
Copy link

flowcool commented Oct 5, 2022

@gaetansnl : Hello, bonjour. Not very used of github but following up closely this change as I want to play with it ;)
Do you know if your change has been merge to "develop" branch or maybe even in "latest" one ?

Thanks,

@tidusjar
Copy link
Member

tidusjar commented Oct 7, 2022

This has not merged as of yet, I'm not sure if I'm wanting to to accept another notification provider, I'm waiting to see if anyone wants this yet as it's another integration that I would need to maintain.

@zaidi-01
Copy link

This has not merged as of yet, I'm not sure if I'm wanting to to accept another notification provider, I'm waiting to see if anyone wants this yet as it's another integration that I would need to maintain.

@tidusjar, any update on this? Would love to see Ntfy as a notification provider.

@DLCIncluded
Copy link

DLCIncluded commented Aug 29, 2023

Would love to see this as well, thanks! Honestly even if it were possible to just add a "custom/other" and allow a custom curl command or script file, as curl is one of the easiest ways to send a ntfy notification

@flowcool
Copy link

@tidusjar : any news from this ? Ntfy is a super lightweight app and self hosted...

@Navino16
Copy link

Navino16 commented Nov 2, 2023

Would love to see this feature merged into Ombi :)

@PoepiePeppie
Copy link

I really don't want to make this a +1 thread, but I would also appreciate this feature being added to the project.
However if you are still against this I would instead ask if the webhook notification feature could be expanded to allow custom json to be sent, so we could use it with ntfy :-D

@pcouy
Copy link

pcouy commented Jul 5, 2024

I would also love to have this feature. I understand the concerns about maintaining one more integration, so I would be satisfied with what others suggested (a way to customize webhooks).

On the other hand, ntfy's API seems to have remained stable over the last few years : the git blame of the documentation for publishing shows that there has been no breaking changes for at least 2 to 3 years.

@sebdanielsson
Copy link

I would like ntfy support as well.

@dkneeland
Copy link

I would also love a ntfy integration!

@dyay108
Copy link

dyay108 commented Oct 4, 2024

piling on as well.

@tidusjar
Copy link
Member

Will be looking at this soon

@zethis
Copy link

zethis commented Dec 2, 2024

Hey any news ? i've tried to setup ntfy in ombi via Webhook, but it's not working :/

ombi     | fail: Ombi.Api.Api[1000]
ombi     |       StatusCode: NotFound, Reason: Not Found, RequestUri:

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 this pull request may close these issues.