Aizat Faiz
01210ce88d
Add url to pushover notification ( #5055 )
...
Auto Test / check-linters (push) Failing after 56s
Auto Test / auto-test (18, ARM64) (push) Has been skipped
Auto Test / auto-test (18, macos-latest) (push) Has been skipped
Auto Test / auto-test (18, ubuntu-latest) (push) Has been skipped
Auto Test / auto-test (18, windows-latest) (push) Has been skipped
Auto Test / auto-test (20.5, ARM64) (push) Has been skipped
Auto Test / auto-test (20.5, macos-latest) (push) Has been skipped
Auto Test / auto-test (20.5, ubuntu-latest) (push) Has been skipped
Auto Test / auto-test (20.5, windows-latest) (push) Has been skipped
Auto Test / armv7-simple-test (18, ARMv7) (push) Has been skipped
Auto Test / armv7-simple-test (20, ARMv7) (push) Has been skipped
Auto Test / e2e-test (push) Has been skipped
CodeQL / Analyze (go) (push) Failing after 27s
CodeQL / Analyze (javascript-typescript) (push) Failing after 21s
Merge Conflict Labeler / Labeling (push) Has been skipped
json-yaml-validate / json-yaml-validate (push) Successful in 8s
Automatically close stale issues / stale (push) Successful in 12s
Co-authored-by: Frank Elsinga <frank@elsinga.de>
2024-08-28 03:24:29 +02:00
Frank Elsinga
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
2024-03-14 14:21:15 +01:00
Matthew Nickson
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>
2023-08-11 15:46:41 +08:00
Louis Lam
343a1d3344
Merge pull request #3203 from CommanderStorm/applied_timezone_formatting
...
chore: Made sure that every notification provider uses `timezone`/`localTime`
2023-05-30 20:36:32 +08:00
Frank Elsinga
f3c09f2bbd
made every Notification provider supply time like dingding after #3152
2023-05-29 19:24:40 +02:00
Maximilian Krauß
a0d0d5b015
fix: sends pushover ttl only if defined
2023-05-26 07:27:43 +02:00
Maximilian Krauß
8d05d80a5f
feat: Adds message ttl to pushover notification
2023-05-26 07:27:43 +02:00
alejandrohernandezrosales
fa23e7ad19
Removed superflous Message prefix
2023-01-20 23:59:11 -06:00
SlothCroissant
f8658d6160
Removed redundant title in Pushover notification
2023-01-08 10:32:16 -06:00
Louis Lam
775d1696fa
Fix pushover device not working #1114
2022-03-24 12:14:17 +08:00
Louis Lam
a864b72e03
fix pushover for general message
2021-11-29 17:19:55 +08:00
Gero Gerke
34b86352f2
remove double spaces
2021-10-05 21:40:59 +02:00
LouisLam
da74391c3e
convert notifications into modules
2021-09-07 22:42:46 +08:00
