Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Summary/Bundle notifications of >1 notification into 1 #1534

Open
1 task done
rob1998 opened this issue Apr 21, 2022 · 2 comments
Open
1 task done

Summary/Bundle notifications of >1 notification into 1 #1534

rob1998 opened this issue Apr 21, 2022 · 2 comments
Labels
area:notifications Everything related to notifications feature-request Request for new features to be added

Comments

@rob1998
Copy link

rob1998 commented Apr 21, 2022

⚠️ Please verify that this feature request has NOT been suggested before.

  • I checked and didn't find similar feature request

🏷️ Feature Request Type

New Notification

🔖 Feature description

W're using the email notification and just received over 100 mails at a time when a server went down.
It would be nice if the notifications could be bundled to a single email.
This could be beneficial for other notification methods as well.

✔️ Solution

In my use case, you could wait 10 sec (or less, maybe make it a setting) before sending an e-mail and check if other notifications are being fired as well.

❓ Alternatives

I have considered other notification methods, but we only ever use e-mail

📝 Additional Context

We have around 400 websites setup 😅

@rob1998 rob1998 added the feature-request Request for new features to be added label Apr 21, 2022
@Computroniks
Copy link
Contributor

Whilst bundling emails could be useful, the implementation would require some quite large changes to the way they are handled currently. Instead of sending notifications when a monitor is detected down, they would have to be added to a queue and then sent after an appropriate interval. This interval could not be too long as not to add an unneccessary delay for urgent notifications. However, it seems in your case that when one system goes down, others go down as well. In this case the dependant monitors PR seems relevant. #1236

@DusterTheFirst
Copy link

This interval could not be too long as not to add an unneccessary delay for urgent notifications.

It might also make sense for this to be a user-configurable option per monitor or globally.

@CommanderStorm CommanderStorm added the area:notifications Everything related to notifications label Dec 7, 2023
@CommanderStorm CommanderStorm changed the title Bundle notifications Summary/Bundle notifications of >1 notification into 1 Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:notifications Everything related to notifications feature-request Request for new features to be added
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants