-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add pending notifications incident history messages #127
Conversation
a65051e
to
60f2f03
Compare
Note that even though I already merged Icinga/icinga-notifications#99, I didn't really test this PR because the current state doesn't work (presumably due to the change channel references to IDs change) and I can't rebase without conflicts (don't look too bad, but I didn't bother resolving them). |
60f2f03
to
8fe79c4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The state must be translatable. The value in the database is not a label.
8fe79c4
to
5c50f4b
Compare
5c50f4b
to
bf026a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wait, there are conflicts
bf026a5
to
be35b80
Compare
No description provided.