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

Change the order of the sections #611

Conversation

jovancacvetkovic
Copy link

@jovancacvetkovic jovancacvetkovic commented May 30, 2023

Description

Sort and reorder the Creation detection rule UX components

Issues Resolved

Resolves #586
Resolves #601
Resolves #613
Resolves #589
Resolves #602

Screenshots

Screenshot from 2023-05-17 16-52-59
Screenshot from 2023-05-17 16-53-17
Screenshot from 2023-05-17 16-53-27
Screenshot from 2023-05-17 16-53-39

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.

Use expression builder instead of code editor

Signed-off-by: Jovan Cvetkovic <[email protected]>
Use expression builder instead of code editor

Signed-off-by: Jovan Cvetkovic <[email protected]>
Use expression builder instead of code editor

Signed-off-by: Jovan Cvetkovic <[email protected]>
Use expression builder instead of code editor

Signed-off-by: Jovan Cvetkovic <[email protected]>
…-analytics-dashboards-plugin into Change_the_order_of_the_sections

# Conflicts:
#	public/app.scss
#	public/pages/Rules/components/RuleEditor/RuleEditorForm.scss
#	public/pages/Rules/components/RuleEditor/components/YamlRuleEditorComponent/RuleTagsComboBox.tsx
Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Jovan Cvetkovic <[email protected]>
@codecov
Copy link

codecov bot commented May 30, 2023

Codecov Report

Merging #611 (07a2d6f) into main (1f8105d) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #611      +/-   ##
==========================================
- Coverage   31.48%   31.33%   -0.16%     
==========================================
  Files         138      139       +1     
  Lines        4335     4446     +111     
  Branches      771      743      -28     
==========================================
+ Hits         1365     1393      +28     
- Misses       2832     2912      +80     
- Partials      138      141       +3     

see 8 files with indirect coverage changes

Signed-off-by: Jovan Cvetkovic <[email protected]>
…ule" page opensearch-project#586

[FEATURE] Improve the Create detection rules - selection panel fields error notifications opensearch-project#601
[FEATURE] Improve the Create detection rules - selection panel condition field is not marked as invalid after submission opensearch-project#613

Signed-off-by: Jovan Cvetkovic <[email protected]>
…ule" page opensearch-project#586

[FEATURE] Improve the Create detection rules - selection panel fields error notifications opensearch-project#601
[FEATURE] Improve the Create detection rules - selection panel condition field is not marked as invalid after submission opensearch-project#613

Signed-off-by: Jovan Cvetkovic <[email protected]>
…-analytics-dashboards-plugin into Improve_list_text_area_UX

# Conflicts:
#	cypress/integration/2_rules.spec.js
#	public/app.scss
#	public/pages/Rules/components/RuleEditor/DetectionVisualEditor.tsx
…acvetkovic/security-analytics-dashboards-plugin into Change_the_order_of_the_sections

# Conflicts:
#	public/pages/Rules/components/RuleEditor/DetectionVisualEditor.tsx
#	public/pages/Rules/components/RuleEditor/RuleEditorForm.tsx
…cacvetkovic/security-analytics-dashboards-plugin into Change_the_order_of_the_sections

# Conflicts:
#	public/pages/Rules/components/RuleEditor/components/YamlRuleEditorComponent/RuleTagsComboBox.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants