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

feat: add UI for chanWebhooks escalation policy step option #2871

Merged
merged 23 commits into from
Mar 20, 2023

Conversation

nataliekorzh
Copy link
Contributor

  • Identified the issue which this PR solves.
  • Read the CONTRIBUTING document.
  • Code builds clean without any errors or warnings.
  • Added appropriate tests for any new functionality.
  • All new and existing tests passed.
  • Added comments in the code, where necessary.
  • Ran make check to catch common errors. Fixed any that came up.

Description:
This PR adds UI support for adding chanWebhooks (webhook notification channels) to escalation policy steps.

Which issue(s) this PR fixes:
Part of #2691
This is a follow-up to #2809

Additional Info:
When the escalation policy step create dialog is opened for the first time, a bad setState() call warning is rendered. This is from the feature flag updating, so it won't be relevant once this PR is approved and the feature flag is removed.

Screenshots:
Screenshot 2023-03-15 at 4 01 06 PM
Screenshot 2023-03-15 at 3 51 20 PM

cmarquis
cmarquis previously approved these changes Mar 16, 2023
@github-actions github-actions bot added size/l and removed size/m labels Mar 16, 2023
@github-actions github-actions bot added size/m and removed size/l labels Mar 16, 2023
cmarquis
cmarquis previously approved these changes Mar 20, 2023
@mastercactapus mastercactapus merged commit 515708c into master Mar 20, 2023
@mastercactapus mastercactapus deleted the epw-select branch March 20, 2023 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants