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

Alert emails should include alert description #1158

Closed
lennartkoopmann opened this issue May 19, 2015 · 3 comments
Closed

Alert emails should include alert description #1158

lennartkoopmann opened this issue May 19, 2015 · 3 comments
Assignees
Milestone

Comments

@lennartkoopmann
Copy link
Member

@lennartkoopmann lennartkoopmann commented May 19, 2015

Our alert emails are not including the nice alert description that the web interface shows. We have that information in a complete string already so adding it to the email should be easy(tm).

Stream received messages matching source:"example.org" (Current grace time: 2 minutes)

@lennartkoopmann lennartkoopmann added this to the 1.2.0 milestone May 19, 2015
@dennisoelkers dennisoelkers self-assigned this May 26, 2015
@lennartkoopmann
Copy link
Member Author

@lennartkoopmann lennartkoopmann commented Jun 1, 2015

Thanks! This seems to only affect the alarm callback and not the normal email though. (I mean the email that users who subscribed to the alert get)

@dennisoelkers
Copy link
Member

@dennisoelkers dennisoelkers commented Jun 1, 2015

No, actually the default email sender when no alarm callback has been defined is using the formatted email sender with a default template/subject (to which I added the alert description now) too.

@lennartkoopmann
Copy link
Member Author

@lennartkoopmann lennartkoopmann commented Jun 1, 2015

Ha! This is also very helpful for #1163

Will check later. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.