-
Notifications
You must be signed in to change notification settings - Fork 41
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
Provide empty states for Findings and Alerts page #494
Provide empty states for Findings and Alerts page #494
Conversation
…h-project#471 Signed-off-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com>
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #494 +/- ##
==========================================
- Coverage 35.62% 35.52% -0.11%
==========================================
Files 119 119
Lines 3253 3257 +4
Branches 521 525 +4
==========================================
- Hits 1159 1157 -2
- Misses 1971 1975 +4
- Partials 123 125 +2 see 5 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
…-analytics-dashboards-plugin into Provide_empty_states_for_Findings_and_Alerts_page
…h-project#471 (opensearch-project#494) Signed-off-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com>
…h-project#471 (opensearch-project#494) (opensearch-project#519) Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
* [FEATURE] Provide empty states for Findings and Alerts page #471 (#494) (#519) Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * fixed recent alerts & finding order; count for pie chart (#576) (#579) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> (cherry picked from commit 3812055) Co-authored-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * Show required field mappings only for enabled rules (#418) (#481) * show required field mappings only for enabled rules Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated mock custom rule to match mapping in backend Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> (cherry picked from commit 1bbc4c5) Co-authored-by: Amardeepsingh Siglani <amardeep7194@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Resolved merge conflicts after cherry-picking PR 418. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * use user provided end date for x-axis calc (#584) (#604) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> (cherry picked from commit c280bad) Co-authored-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
…h-project#471 (opensearch-project#494) (opensearch-project#519) Signed-off-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com> Co-authored-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
* [FEATURE] Provide empty states for Findings and Alerts page #471 (#494) (#519) Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * fixed recent alerts & finding order; count for pie chart (#576) (#579) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> (cherry picked from commit 3812055) Co-authored-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * Show required field mappings only for enabled rules (#418) (#481) * show required field mappings only for enabled rules Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> * updated mock custom rule to match mapping in backend Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> (cherry picked from commit 1bbc4c5) Co-authored-by: Amardeepsingh Siglani <amardeep7194@gmail.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Resolved merge conflicts after cherry-picking PR 418. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * use user provided end date for x-axis calc (#584) (#604) Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com> (cherry picked from commit c280bad) Co-authored-by: Amardeepsingh Siglani <amardeep7194@gmail.com> --------- Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: Amardeepsingh Siglani <amardeep7194@gmail.com> cr: https://code.amazon.com/reviews/CR-93561817
Description
Adds empty state messages for the Findings and Alerts page.
Find all the mocks attached on issue #471
Issues Resolved
Resolves #471
Screenshots
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.