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
Enhance notification types #137
Conversation
4617a5c
to
a8cf6f4
Compare
a8cf6f4
to
f88cc25
Compare
ae322aa
to
f99810b
Compare
1098d13
to
63c840e
Compare
63c840e
to
1127399
Compare
97824ac
to
86c2bc0
Compare
2fb2aff
to
82bff48
Compare
b7a76cc
to
28cbafe
Compare
38b63c0
to
f193809
Compare
c581ca5
to
604c883
Compare
case TypeState: | ||
case TypeAcknowledgement: | ||
case TypeCommentAdded: | ||
case TypeCommentRemoved: | ||
case TypeDowntimeStart: | ||
case TypeDowntimeEnd: | ||
case TypeDowntimeCancelled: | ||
case TypeCustom: | ||
case TypeFlappingStart: | ||
case TypeFlappingEnd: | ||
return nil |
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.
There's no automatic fallthrough between cases, so this doesn't really do what you probably expect (or in some way it does, the next statement after switch
will be the return nil
at the end of the function, so you won't notice a difference here.
Hint: you can do case A, B, C:
bb764bd
to
d6c4d36
Compare
I'll take over this PR from now on! |
..because it is used for both incident and non-incident states
…istory.go` Move `HistoryRow` struct to this file, as history table has also been generalised
71a6b2b
to
ddad1e4
Compare
resolves #106