3 Commits (561d307fb8caab156ae5165c31a7a3a8ed314f04)

Author	SHA1	Message	Date
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
Daeho Ro	5f6347d277	pull request for adding alertnow notification	2022-06-12 04:02:44 +09:00