-
Notifications
You must be signed in to change notification settings - Fork 91
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
Implemented UX support for configuring doc level monitors. #218
Implemented UX support for configuring doc level monitors. #218
Conversation
Signed-off-by: AWSHurneyt <[email protected]>
The unit test flow is currently failing due to outdated snapshots.
A solution seems to involve adding the following to the top of each test class. There are over 100 test classes at the moment, so that annotation will be added in a separate PR.
|
public/components/Flyout/flyouts/components/AlertsDashboardFlyoutComponent.js
Show resolved
Hide resolved
public/components/Flyout/flyouts/components/AlertsDashboardFlyoutComponent.js
Show resolved
Hide resolved
public/components/Flyout/flyouts/components/AlertsDashboardFlyoutComponent.js
Show resolved
Hide resolved
public/pages/CreateMonitor/containers/CreateMonitor/utils/formikToMonitor.js
Show resolved
Hide resolved
Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: Ashish Agrawal <[email protected]>
…tyle of perform action section of Action element. Signed-off-by: AWSHurneyt <[email protected]>
57de058
Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #218 +/- ##
==========================================
- Coverage 56.79% 53.12% -3.68%
==========================================
Files 198 208 +10
Lines 4756 5207 +451
Branches 651 704 +53
==========================================
+ Hits 2701 2766 +65
- Misses 2053 2439 +386
Partials 2 2
Continue to review full report at Codecov.
|
Signed-off-by: AWSHurneyt [email protected]
Description
Refactored existing UX elements to support configuring doc level monitors.
Screenshots
Visual editor UX
Extraction query editor UX
Monitor details page
Alerts by trigger flyout, and finding flyout
Issues Resolved
[List any issues this PR will resolve]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.