Bulk edit monitor notification settings

[gorkem-bwl]

When the user has several monitors, it should be easy to bulk edit them. I can work with the contributor together on how to implement this on the UI.

[karins28]

I'd love to take this on!

For the frontend, I'm thinking of extending the existing notification dialog by adding a multi-select list of monitors for each notification channel (Slack, Discord, etc.). This could be done either by introducing an additional prop or by extending the current class—happy to go with whichever approach makes more sense to you.

Does this task also require backend changes? I noticed there's already some existing logic inplace including a queue and worker mechanism.

[gorkem-bwl]

I'd love to take this on!

For the frontend, I'm thinking of extending the existing notification dialog by adding a multi-select list of monitors for each notification channel (Slack, Discord, etc.). This could be done either by introducing an additional prop or by extending the current class—happy to go with whichever approach makes more sense to you.

Does this task also require backend changes? I noticed there's already some existing logic inplace including a queue and worker mechanism.

Sure, it has the BE part as well. So you'd need to think holistically, and come up with details about how to implement this.

[karins28]

@gorkem-bwl
Interesting task! I'm thinking of using a batching/queuing approach

Regarding the api:
1.I'm thinking about adding a new api: updateMonitors (e instead of multiple api calls to /update-monitor
input: (monitor ids / monitor type)
using mongodb's updateMany function

2.(Optional) Regarding the notifications implementation :
I noticed that the current notification logic in JobQueueHelper sends each notification individually asynchronically . I'm considering introducing a notificationQueue that would allow batching of notifications sharing the same webhook. This would enable grouped delivery at defined intervals, reducing noise and improving efficiency.

let me know your thoughts!

[gorkem-bwl]

@ajhollid fyi