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(notifications): implement gotify provider #1081

Open
wants to merge 4 commits into
base: canary
Choose a base branch
from

Conversation

depado
Copy link

@depado depado commented Jan 10, 2025

image

image

This PR attempts to implement gotify as a notification provider.

  • Add/delete/edit gotify notification
  • Implement event notifications (deploy, build error, database backup, dokploy restart, docker cleanup)

@vishalkadam47
Copy link
Contributor

Did you check apprise, take a look if that's useful
https://github.com/caronc/apprise

@depado
Copy link
Author

depado commented Jan 12, 2025

I didn't but since gotify implementation is basically as simple as an HTTP post, I think it will work fine as is. That being said, there could be a more generic way to implement notifications in the future using apprise I guess 😁

@depado depado force-pushed the gotify-notifications branch from 1bdcde1 to 62a5eba Compare January 13, 2025 17:58
@depado depado force-pushed the gotify-notifications branch from 62a5eba to 2e4f488 Compare January 13, 2025 18:03
@depado depado marked this pull request as ready for review January 13, 2025 18:04
@depado depado requested a review from Siumauricio as a code owner January 13, 2025 18:04
@Siumauricio
Copy link
Contributor

How can I test this? If you can add a video would be awesome!

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.

3 participants