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

Simplify detector creation UX #738

Conversation

amsiglan
Copy link
Collaborator

@amsiglan amsiglan commented Oct 4, 2023

Description

This PR simplified detector creation workflow by:

  • Eliminating Review step
  • Adding default values in configure alert's trigger condition to make it faster
  • Refactoring field mapping UI to make it more intuitive
  • Adding new entry points in the Overview page
  • Reducing clutter in getting started pop up

Check List

  • Commits are signed per the DCO using --signoff

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.

@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

Merging #738 (1e1cdf2) into main (87bcabb) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #738   +/-   ##
=======================================
  Coverage   31.48%   31.49%           
=======================================
  Files         150      149    -1     
  Lines        4818     4826    +8     
  Branches      858      867    +9     
=======================================
+ Hits         1517     1520    +3     
- Misses       3122     3123    +1     
- Partials      179      183    +4     

see 15 files with indirect coverage changes

Signed-off-by: Amardeepsingh Siglani <[email protected]>
@AWSHurneyt AWSHurneyt merged commit 8a7bf7d into opensearch-project:main Oct 4, 2023
9 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 4, 2023
* removed review page; enhanced alerts page

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated tests

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated trigger details subheading to use selected data

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated define detector, overview pages

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* reverting cypress workflow

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated field mapping texts

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 8a7bf7d)
@amsiglan amsiglan changed the title [Part 2] Simplify detector creation UX Simplify detector creation UX Oct 4, 2023
AWSHurneyt pushed a commit that referenced this pull request Oct 4, 2023
* removed review page; enhanced alerts page

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated tests

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated trigger details subheading to use selected data

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated define detector, overview pages

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* reverting cypress workflow

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated field mapping texts

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 8a7bf7d)

Co-authored-by: Amardeepsingh Siglani <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants