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

Email Notifications #3191

Closed
1 task done
evdev opened this issue May 24, 2023 · 4 comments · Fixed by #3414
Closed
1 task done

Email Notifications #3191

evdev opened this issue May 24, 2023 · 4 comments · Fixed by #3414
Labels
feature-request Request for new features to be added

Comments

@evdev
Copy link

evdev commented May 24, 2023

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

  • I checked and didn't find similar feature request

🏷️ Feature Request Type

UI Feature

🔖 Feature description

Can you please enhance the email notification option to allow customizing the body of the email?
Would it be possible to add a variable for email addresses for each monitor and to send a notification to the email addresses in the variable? This way I won't need to configure a notification type for each email address I would like to send to and it can instead send the email automatically to the listed email addresses in the local variable of the monitored entity.

✔️ Solution

add a customization field for the body
add variable to each monitor and allow the variable to be used for the monitor's notifications

❓ Alternatives

No response

📝 Additional Context

No response

@evdev evdev added the feature-request Request for new features to be added label May 24, 2023
@prohand
Copy link

prohand commented May 24, 2023

Hello, great feature :)
hope it is implemented

@CommanderStorm
Copy link
Collaborator

@evdev
Customising the body of mails is tracked in #1529, please don't post duplicate issues, as this makes issue management harder.

Handling emails like this would be very messy. Not everybody wants email notifications.
=> I don't think Uptime Kuma should modify the Notification system like this.
If you really need this feature, you could achieve this with other email filtering tools like Sieve.

Please go into detail about your intended use case, if you really need this.

@CommanderStorm
Copy link
Collaborator

@prohand
Please refrain from posting +1 / requests for updates things on issues, as this makes issue-management harder.
Issues are for discussing what needs to be done how by whom.
We use 👍🏻 on issues to prioritise work.

@evdev
Copy link
Author

evdev commented May 24, 2023

@CommanderStorm

Thanks for your prompt response. I will look into Sieve.

The reason I am asking for this feature is because my use case is probably a bit different than the typical IT usage where you are monitoring internal and cloud based systems for which only a few stakeholders are responsible. In my case the monitored devices are spread out across many sites in clients homes. If the device is down it is probably due to an internet or power issue which needs to be resolved by the contact on site. That is why I would prefer that the emails go directly to the one responsible for it. The way it is setup now, to accomplish this I need to setup a notification with all of the SMTP info for each recipient even though they are only monitoring a single device. I hope that helps.

While this might not be the most common use case I don't think it is so fringe. The way it is setup now assumes that notifications will only be going out to a few people, but sometimes even with one organization the person being notified may only need to know about the uptime of 1 or 2 entities. That is why I am suggesting that the credentials for authentication be stored separately from the intended recipient. So this would apply to phone numbers, emails addresses, handles etc. To preserve current functionality perhaps the currently configured recipient can be configured as the default recipient unless a different one is specified with the monitored entity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features to be added
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants