-
Notifications
You must be signed in to change notification settings - Fork 309
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
Pausing webhooks #7401
Pausing webhooks #7401
Conversation
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.
Here is some UI fixes that can be done:
@KrishnaIyer if you think of a better text for the modal let us know
Text for Text above |
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
a9c2f9f
to
0ab69d1
Compare
0ab69d1
to
fa89363
Compare
Summary
Reference #7223
Changes
Testing
Steps
See issue for more details on creating an active webhook
For this task
Create a webhook
Open webhook edit form for example
/console/applications/abc/integrations/webhooks/pausing-webhooks
Go to the bottom of the page and click the pause icon, the modal should open and then click
Pause webhook
Try reactivating the webhook using either the notification at the top of the edit form or using the play button at the bottom of the page.
In each state you can see if your webhook if getting to it destination and also check the status on the list view
/console/applications/abc/integrations/webhooks
Results
Pause webhook
Activate webhook
List view
Health status enabled
Health status disabled
Regressions
...
Notes for Reviewers
Checklist
README.md
for the chosen target branch.CHANGELOG.md
.CONTRIBUTING.md
, there are no fixup commits left.