a9a1cf1353
Chore: General notification reformatting ( #3182 )
...
- I unified where in file the name of `NotificationProvider.name` is placed
- I made sure that all the providers adhere to the signature of `NotificationProvider.send()`
- I made sure that all the providers use `okMsg` if returning success messages directly from the function.
Here a discussion should be had:
Should this be refactored into a constant of `NotificationProvider`? I could imagine that `NotificationProvider.SENDING_SUCCESSFULL` could be a suitable alternative.
- I made sure all providers have the URL they `POST`/`GET` to be extraced into a variable.
=> refactored this way due to Nelsons suggestion
8 months ago
a13fc7079e
Merge branch 'master' into 2.0.X
...
# Conflicts:
# package-lock.json
1 year ago
c0e273df5b
Show hostname:port for gamedig monitor on Discord notification ( #3643 )
1 year ago
8a92054c2b
Added JSDoc to ESLint ( #3529 )
...
* Added JSDoc to eslint rules
Signed-off-by: Matthew Nickson <mnickson@sidingsmedia.com>
* Fixed JSDoc eslint errors
Signed-off-by: Matthew Nickson <mnickson@sidingsmedia.com>
* Update the check-linters workflow to Node.js 20
---------
Signed-off-by: Matthew Nickson <mnickson@sidingsmedia.com>
Co-authored-by: Louis Lam <louislam@users.noreply.github.com>
1 year ago
f3c09f2bbd
made every Notification provider supply time like dingding after #3152
1 year ago
8bb064c6fa
Merge pull request #2157 from Mikkel-T/fix-discord-embed
...
Improve the URL field in Discord embeds
2 years ago
f32441e2f6
fix discord notification not sending when docker container is down
2 years ago
a42f7416b5
Improve the URL field in Discord embeds
...
Instead of having two different ways of showing the URL field in Discord embeds, always show the raw address.
2 years ago
f23baf9c22
Added Push Monitor to Discord Notifications
2 years ago
cec0521834
[Discord] Fix ping type should no port, update better naming
3 years ago
c5faf709b8
Merge branch 'master' into fix-1448-discord-service-url
3 years ago
47c72192e1
[eslint] Enable yoda and eqeqeq
3 years ago
288ed1e3ca
Merge branch 'master' into fix-1448-discord-service-url
3 years ago
649f3106e1
Enforce semicolon, fix format globally
3 years ago
2638d68c97
Cover dns and steam types in Discord notifs
3 years ago
0961c6d9b3
Check for ping and port type in discord notifs
3 years ago
34b86352f2
remove double spaces
3 years ago
8352d9abbe
add posibility to have a prefixMessage before the embed
3 years ago
da74391c3e
convert notifications into modules
3 years ago
Frank Elsinga
Louis Lam
Lior Slakman
Matthew Nickson
zImPatrick
Mikkel-T
DasCanard
Jordan Bertasso
Gero Gerke
Denis Freund