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

Remove native notification services #3243

Open
cafferata opened this issue Dec 12, 2024 · 3 comments
Open

Remove native notification services #3243

cafferata opened this issue Dec 12, 2024 · 3 comments

Comments

@cafferata
Copy link
Collaborator

I wanted to propose removing the native notification services (Slack/Mattermost, Microsoft Teams, Telegram) in favor of fully transitioning to Shoutrrr. Shoutrrr provides broader support and flexibility, making it a more streamlined and unified solution for notifications. Let me know what you think!

@tlimoncelli
Copy link
Contributor

YES!

Take it a bit further:

  1. creds.js shouldn't need notifications -> shouterrr_url. There should be a "shouterrr" option that just takes a list of shouterrr URLs.
  2. Native dnscontrol notifications should rewrite the URL into a shouterrr URL so we can delete the old code.

@cafferata
Copy link
Collaborator Author

Originally posted by @tlimoncelli in #3242 (comment)

If there are any methods that are supported by both the native dnscontrol and Shoutrrr, let's mark the native one as "legacy" and recommend the Shoutrrr version in the future. (In fact, maybe in the future the native methods should just call Shoutrrr?)

@hmoffatt
Copy link
Contributor

Sounds good to me.

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

No branches or pull requests

3 participants