-
Notifications
You must be signed in to change notification settings - Fork 257
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 acknowledged, resolved user information on webhook payload #4176
Add acknowledged, resolved user information on webhook payload #4176
Conversation
I have added the details separately outside of the alertgroup since we use the same alert group serializer for the public API response. For integration details also we have followed the similar pattern. |
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.
LGTM 👍 thanks again for your continued contributions @ravishankar15
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.
actually, one last minor ask. Do you mind updating the documentation here?
once that's squared away I will go ahead and merge ✌️
Merge queue setting changed
5301067
to
d417b80
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.
Looks good, thanks!
What this PR does
Adds acknowledged and resolved user information to the web hook payload
Which issue(s) this PR closes
Closes #4126
Checklist
pr:no public docs
PR label added if not required)release:
). These labels dictate how your PR willshow up in the autogenerated release notes.