vvanouytsel / jetspotter

MIT License
22 stars 2 forks source link

Feature request: gotify notifications #12

Closed visibilityspots closed 1 year ago

visibilityspots commented 1 year ago

very interested to use jetspotter, since I unified all my notifications towards https://gotify.net/ I tried to write a notification based on https://github.com/gotify/go-api-client but can't get it to work due to lack of go and time :(

The example code I got from the go-api-client is currently failing on

err declared and not used
and a 
missing return

If you can point me in a direction regarding it let me know!

vvanouytsel commented 1 year ago

Haven't heard of gotify yet. I will do some investigation and see if I can get it to work. 👍

vvanouytsel commented 1 year ago

@visibilityspots I have added support for Gotify notifications in 1.19. Could you test it out and verify that it works as you expect? I tested it locally by running gotify in a container and that seemed to work.

https://vvanouytsel.github.io/jetspotter/#gotify

visibilityspots commented 1 year ago

works as expected on my local machine, will try to have it running on my nomad cluster as a next step ;)