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

Use incident severity in notifications #127

Closed
julianbrost opened this issue Nov 20, 2023 · 0 comments · Fixed by #134
Closed

Use incident severity in notifications #127

julianbrost opened this issue Nov 20, 2023 · 0 comments · Fixed by #134
Assignees

Comments

@julianbrost
Copy link
Collaborator

Currently, the notification code seems to use the event severity in places where it probably should use the incident severity instead. So far, this was no problem as pretty much every notification was a direct result of an incoming state event, but it will become a problem with other notification types (#106). This resulted in workaround like in 1acb259 which should not be necessary.

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

Successfully merging a pull request may close this issue.

3 participants