Implemented a toast to display successful attempts to acknowledge alerts. Refactored alerts dashboard flyout to refresh its alerts table when alerts are acknowledged. #160

AWSHurneyt · 2021-12-10T00:04:36Z

Signed-off-by: AWSHurneyt hurneyt@amazon.com

Description

Closed the original PR for this implementation.

Refactored AlertsDashboardFlyoutComponent::getBucketLevelGraphConditions to return a string with line breaks instead of an array of HTML elements.
Refactored alerts dashboard flyout to refresh its alerts table when alerts are acknowledged.
Implemented a toast to display successful attempts to acknowledge alerts.
Implemented unit tests for various helper methods.
Implemented integration tests for the alerts dashboard flyout.

Issues Resolved

#125

Check List

New functionality includes testing.
- All tests pass
New functionality has been documented.
- New functionality has javadoc added
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.

…actored AlertsDashboardFlyoutComponent::getBucketLevelGraphConditions to return a string with line breaks instead of an array of HTML elements. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

codecov-commenter · 2021-12-18T01:37:22Z

Codecov Report

❗ No coverage uploaded for pull request base (main@d01afa1). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #160   +/-   ##
=======================================
  Coverage        ?   49.24%           
=======================================
  Files           ?      202           
  Lines           ?     4870           
  Branches        ?      601           
=======================================
  Hits            ?     2398           
  Misses          ?     2470           
  Partials        ?        2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d01afa1...d545434. Read the comment docs.

public/pages/Dashboard/utils/helpers.test.js

public/components/Flyout/flyouts/components/AlertsDashboardFlyoutComponent.js

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

cypress/integration/alerts_dashboard_flyout_spec.js

annie3431 · 2022-01-11T22:44:16Z

public/pages/Dashboard/utils/helpers.test.js

+          ACKNOWLEDGED: 0,
+          ERROR: 0,
+          total: 2,
+          alerts: [


Seems like expectedOutput can be defined using alerts, right? Would be good to reuse constants here and reduce duplicate lines of code.

The alerts can be defined as constants, then in individual tests fill them the alerts array

Just to confirm, you're saying refactor the declaration of expectedOutput on line 159 to something like what I pasted below, correct?

const alerts = *list of alerts*; const expectedOutput = [ { ACTIVE: 2, ACKNOWLEDGED: 0, ERROR: 0, total: 2, alerts: alerts ...

If so, I don't think that would accomplish what we're looking for with this test. groupAlertsByTrigger takes a list of unsorted alerts, sorts them based on their respective trigger, and returns a list of objects (1 for each trigger) that includes only the alerts associated with that trigger.

I see. Thanks for clarifying!

… periods. Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

…rds-plugin into issue125-main

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

…g-dashboards-plugin into issue125-main

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

…rts. Refactored alerts dashboard flyout to refresh its alerts table when alerts are acknowledged. (opensearch-project#160) * Implemented unit and integ tests for the alerts dashboard flyout. Refactored AlertsDashboardFlyoutComponent::getBucketLevelGraphConditions to return a string with line breaks instead of an array of HTML elements. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Removed an unused test variable. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Removed debug logs. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Implemented unit test. Refactored integration tests to use fewer wait periods. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Examining flakiness in cypress test. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Added short wait period to flyout cypress tests to alleviate flakiness. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored flyout cypress tests to use aliases. Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

…rts. Refactored alerts dashboard flyout to refresh its alerts table when alerts are acknowledged. (opensearch-project#160) * Implemented unit and integ tests for the alerts dashboard flyout. Refactored AlertsDashboardFlyoutComponent::getBucketLevelGraphConditions to return a string with line breaks instead of an array of HTML elements. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Removed an unused test variable. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Removed debug logs. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Implemented unit test. Refactored integration tests to use fewer wait periods. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Examining flakiness in cypress test. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Added short wait period to flyout cypress tests to alleviate flakiness. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Refactored flyout cypress tests to use aliases. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com>

…rts. Refactored alerts dashboard flyout to refresh its alerts table when alerts are acknowledged. (opensearch-project#160) * Implemented unit and integ tests for the alerts dashboard flyout. Refactored AlertsDashboardFlyoutComponent::getBucketLevelGraphConditions to return a string with line breaks instead of an array of HTML elements. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Removed an unused test variable. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Removed debug logs. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Implemented unit test. Refactored integration tests to use fewer wait periods. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Examining flakiness in cypress test. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Added short wait period to flyout cypress tests to alleviate flakiness. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Refactored flyout cypress tests to use aliases. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

* OpenSearch 1.1.0 release (#103) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update comments Signed-off-by: Annie Lee <leeyun@amazon.com> * Update version in package.json (#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Text updates (#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Updates to workflow, unit tests, and some appearance (#114) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <leeyun@amazon.com> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <leeyun@amazon.com> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <leeyun@amazon.com> Co-authored-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Cherry-pick commits from main branch to 1.x branch (#131) * Bumps version to 1.2 (#128) * Bumps version to 1.2 * Changes test workflows to follow Dashboards 1.x Signed-off-by: Clay Downs <downsrob@amazon.com> * Fixed a bug that displayed all alerts for a monitor on individual triggers' flyouts. Fixed a bug that displayed incorrect source for the condition field on the alerts flyout. Fixed a bug that displayed incorrect severity on the alerts flyout. Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Fixed bug relating to validation of popovers when defining monitor queries. (#123) (#130) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update comments Signed-off-by: Annie Lee <leeyun@amazon.com> * Update version in package.json (#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Text updates (#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <leeyun@amazon.com> * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <leeyun@amazon.com> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <leeyun@amazon.com> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <leeyun@amazon.com> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <leeyun@amazon.com> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <leeyun@amazon.com> * Fixed a few bugs * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Fixed a bug that displayed all alerts for a monitor on individual trigger flyouts. Fixed a bug that diplayed incorrect source for the condition field on the alerts flyout. Fixed a bug that diplaying incorrect severity on the alerts flyout. * Updated release notes to reflect PR 122 bug fix. * Fixing number of alerts displayed on Monitors tab. * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * More bug fix Signed-off-by: Annie Lee <leeyun@amazon.com> * Skip test based on modification Signed-off-by: Annie Lee <leeyun@amazon.com> * Skip test based on modification Signed-off-by: Annie Lee <leeyun@amazon.com> * Update popover windows to remove item when filed is not defined * Update field validation * Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Removed experimental code and comments. * Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix * Update Dashboard.js * Support data filter when using null operator * Update WhereExpression.js * Fixed a bug that was causing incorrect pagination display on alerts flyout. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Update metric error for query monitors * Update MetricExpression.js * Removed experimental dev code. * Updated release notes. Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: Clay Downs <downsrob@amazon.com> Co-authored-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Cherry-picking commits from main to 1.x branch for 1.2 release (#142) * Fixes flaky test and removes local publishing of plugin dependencies (#135) * Fixes 'bucket level monitor can be created by extraction query' flaky test * Removes local publishing of plugin dependencies for github cypress tests Signed-off-by: Clay Downs <downsrob@amazon.com> * Update copyright notice (#140) Signed-off-by: Mohammad Qureshi <qreshi@amazon.com> * Added 1.2 release notes. (#141) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Added 1.2 release notes. Signed-off-by: Thomas Hurney <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Added 1.2 release notes. Signed-off-by: Thomas Hurney <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Updated 1.2 release notes. Signed-off-by: Thomas Hurney <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: Clay Downs <downsrob@amazon.com> Co-authored-by: Mohammad Qureshi <qreshi@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Added DCO section to CONTRIBUTING.md and GitHub workflows. Updated copyright notice. Signed-off-by: Thomas Hurney <hurneyt@amazon.com> (#145) Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * support creating monitor for anomaly detector with custom result index (#143) (#147) * support creating monitor for anomaly detector with custom result index Signed-off-by: Yaliang Wu <ylwu@amazon.com> * add release note Signed-off-by: Yaliang Wu <ylwu@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Bumping version to 1.3. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> (#159) Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * initial commit (#150) Signed-off-by: CEHENKLE <henkle@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Add .whitesource configuration file (#153) Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Implemented a toast to display successful attempts to acknowledge alerts. Refactored alerts dashboard flyout to refresh its alerts table when alerts are acknowledged. (#160) * Implemented unit and integ tests for the alerts dashboard flyout. Refactored AlertsDashboardFlyoutComponent::getBucketLevelGraphConditions to return a string with line breaks instead of an array of HTML elements. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Removed an unused test variable. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Removed debug logs. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Implemented unit test. Refactored integration tests to use fewer wait periods. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Examining flakiness in cypress test. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Added short wait period to flyout cypress tests to alleviate flakiness. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Refactored flyout cypress tests to use aliases. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Adding basic unit tests (#151) * Add flyout render Signed-off-by: Annie Lee <leeyun@amazon.com> * Add some basic tests Signed-off-by: Annie Lee <leeyun@amazon.com> * Add more tests Signed-off-by: Annie Lee <leeyun@amazon.com> * Update jest.config.js Signed-off-by: Annie Lee <leeyun@amazon.com> * Add tests Signed-off-by: Annie Lee <leeyun@amazon.com> * Update dashboard sample alerttime Signed-off-by: Annie Lee <leeyun@amazon.com> * Update dashboard test alert time snapshot Signed-off-by: Annie Lee <leeyun@amazon.com> * Update Dashboard.test.js Signed-off-by: Annie Lee <leeyun@amazon.com> * Remove test alert start time Signed-off-by: Annie Lee <leeyun@amazon.com> * Update VisualGraph test and code Signed-off-by: Annie Lee <leeyun@amazon.com> * Update package.json to run unit tests in UTC timezone Signed-off-by: Annie Lee <leeyun@amazon.com> * Update dashboard sample alert start time Signed-off-by: Annie Lee <leeyun@amazon.com> * Update Dashboard test Signed-off-by: Annie Lee <leeyun@amazon.com> * Update Dashboard.test.js.snap Signed-off-by: Annie Lee <leeyun@amazon.com> * Update snapshot file Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Fix the error handling when config index is not found (#173) Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Updated copyright notices and headers. (#168) * Updated CONTRIBUTING.md and NOTICE files. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Updated license headers. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored acknowledge alerts button on Alerts by trigger dashboard page to be a modal experience. (#167) * Refactored Acknowledge button on Alerts by trigger dashboard to display a modal. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Updated license headers. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Adding a few more basic unit tests (#180) * Add alerting dashboards flyout render unit test Signed-off-by: Annie Lee <leeyun@amazon.com> * Add MonitorTimeFields validation tests Signed-off-by: Annie Lee <leeyun@amazon.com> * Refactor error messages to add period Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Remove node version declaration in package.json (#166) * Remove node version declaration in package.json Signed-off-by: Annie Lee <leeyun@amazon.com> * Remove react-router-dom version declaration Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Update DestinationsService.js (#182) Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Add backport workflow (#176) * Add backport workflow Signed-off-by: Annie Lee <leeyun@amazon.com> * Create delete_backport_branch.yml Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Configure test workflows to run on 1.x branches (#183) Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Add 1.3 release notes (#175) (#185) * Create opensearch-alerting-dashboards-plugin.release-notes-1.3.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.3.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> (cherry picked from commit 98f7904) Co-authored-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Add backport documentation link (#184) (#187) * Create BACKPORT.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Replace documentation with a link Signed-off-by: Annie Lee <leeyun@amazon.com> (cherry picked from commit 0a0c8c5) Co-authored-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Implemented support for cluster metrics monitors (#162) (#189) * Implemented LocalUriInput component, and supporting methods. Implemented unit and integration tests for LocalUriInput. Refactored various other classes and components to support LocalUriInput. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored learn more links to open new tabs. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Implemented default trigger conditions for API supported by the LocalUriInput feature. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored button text and size based on UX reviewer feedback. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Added documentation link for cluster stats API. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Adjusted some wording based on UX reviewer feedback. Implemented modal that displays when changing request type. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored feature assets to use ClusterMetrics naming convention instead of LocalUriInput. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored behavior of cluster metrics clear triggers modal, and implemented tests. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Added periods to error messages. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Removed fixed width for clearTriggersModal. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored the cluster metrics radio card to be a monitor type instead of monitor definition type. Refactored request type selection combobox so it's not clearable, and reworded the clearTriggersModal text and refactored tests accordingly. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Updated copyright headers. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactoring monitorType checking logic to accommodate cluster metrics monitors. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored help text to use lowercase letters. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Implemented a helper method as a temporary solution to some incorrect formatting of cluster metrics monitors returned by the getMonitors API. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored cluster metrics feature to remove support for CAT repositories. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored cluster metrics request types labels to no longer reference CAT. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Adjusted the example trigger conditions for two request types. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: Annie Lee <71157062+leeyun-amzn@users.noreply.github.com> Co-authored-by: Clay Downs <downsrob@amazon.com> Co-authored-by: Mohammad Qureshi <qreshi@amazon.com> Co-authored-by: Yaliang <49084640+ylwu-amzn@users.noreply.github.com> Co-authored-by: CEHENKLE <henkle@amazon.com> Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>

* Add backport documentation link (#184) (#187) * Create BACKPORT.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Replace documentation with a link Signed-off-by: Annie Lee <leeyun@amazon.com> (cherry picked from commit 0a0c8c5) Co-authored-by: Annie Lee <leeyun@amazon.com> * 1.3release - cherry-picking from main to 1.x branch (#192) * OpenSearch 1.1.0 release (#103) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update comments Signed-off-by: Annie Lee <leeyun@amazon.com> * Update version in package.json (#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Text updates (#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Updates to workflow, unit tests, and some appearance (#114) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <leeyun@amazon.com> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <leeyun@amazon.com> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <leeyun@amazon.com> Co-authored-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Cherry-pick commits from main branch to 1.x branch (#131) * Bumps version to 1.2 (#128) * Bumps version to 1.2 * Changes test workflows to follow Dashboards 1.x Signed-off-by: Clay Downs <downsrob@amazon.com> * Fixed a bug that displayed all alerts for a monitor on individual triggers' flyouts. Fixed a bug that displayed incorrect source for the condition field on the alerts flyout. Fixed a bug that displayed incorrect severity on the alerts flyout. Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Fixed bug relating to validation of popovers when defining monitor queries. (#123) (#130) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update comments Signed-off-by: Annie Lee <leeyun@amazon.com> * Update version in package.json (#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Text updates (#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <leeyun@amazon.com> * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <leeyun@amazon.com> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <leeyun@amazon.com> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <leeyun@amazon.com> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <leeyun@amazon.com> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <leeyun@amazon.com> * Fixed a few bugs * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Fixed a bug that displayed all alerts for a monitor on individual trigger flyouts. Fixed a bug that diplayed incorrect source for the condition field on the alerts flyout. Fixed a bug that diplaying incorrect severity on the alerts flyout. * Updated release notes to reflect PR 122 bug fix. * Fixing number of alerts displayed on Monitors tab. * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * More bug fix Signed-off-by: Annie Lee <leeyun@amazon.com> * Skip test based on modification Signed-off-by: Annie Lee <leeyun@amazon.com> * Skip test based on modification Signed-off-by: Annie Lee <leeyun@amazon.com> * Update popover windows to remove item when filed is not defined * Update field validation * Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Removed experimental code and comments. * Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix * Update Dashboard.js * Support data filter when using null operator * Update WhereExpression.js * Fixed a bug that was causing incorrect pagination display on alerts flyout. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Update metric error for query monitors * Update MetricExpression.js * Removed experimental dev code. * Updated release notes. Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: Clay Downs <downsrob@amazon.com> Co-authored-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Cherry-picking commits from main to 1.x branch for 1.2 release (#142) * Fixes flaky test and removes local publishing of plugin dependencies (#135) * Fixes 'bucket level monitor can be created by extraction query' flaky test * Removes local publishing of plugin dependencies for github cypress tests Signed-off-by: Clay Downs <downsrob@amazon.com> * Update copyright notice (#140) Signed-off-by: Mohammad Qureshi <qreshi@amazon.com> * Added 1.2 release notes. (#141) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Added 1.2 release notes. Signed-off-by: Thomas Hurney <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Added 1.2 release notes. Signed-off-by: Thomas Hurney <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Updated 1.2 release notes. Signed-off-by: Thomas Hurney <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: Clay Downs <downsrob@amazon.com> Co-authored-by: Mohammad Qureshi <qreshi@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Added DCO section to CONTRIBUTING.md and GitHub workflows. Updated copyright notice. Signed-off-by: Thomas Hurney <hurneyt@amazon.com> (#145) Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * support creating monitor for anomaly detector with custom result index (#143) (#147) * support creating monitor for anomaly detector with custom result index Signed-off-by: Yaliang Wu <ylwu@amazon.com> * add release note Signed-off-by: Yaliang Wu <ylwu@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Bumping version to 1.3. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> (#159) Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * initial commit (#150) Signed-off-by: CEHENKLE <henkle@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Add .whitesource configuration file (#153) Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Implemented a toast to display successful attempts to acknowledge alerts. Refactored alerts dashboard flyout to refresh its alerts table when alerts are acknowledged. (#160) * Implemented unit and integ tests for the alerts dashboard flyout. Refactored AlertsDashboardFlyoutComponent::getBucketLevelGraphConditions to return a string with line breaks instead of an array of HTML elements. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Removed an unused test variable. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Removed debug logs. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Implemented unit test. Refactored integration tests to use fewer wait periods. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Examining flakiness in cypress test. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Added short wait period to flyout cypress tests to alleviate flakiness. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Refactored flyout cypress tests to use aliases. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Adding basic unit tests (#151) * Add flyout render Signed-off-by: Annie Lee <leeyun@amazon.com> * Add some basic tests Signed-off-by: Annie Lee <leeyun@amazon.com> * Add more tests Signed-off-by: Annie Lee <leeyun@amazon.com> * Update jest.config.js Signed-off-by: Annie Lee <leeyun@amazon.com> * Add tests Signed-off-by: Annie Lee <leeyun@amazon.com> * Update dashboard sample alerttime Signed-off-by: Annie Lee <leeyun@amazon.com> * Update dashboard test alert time snapshot Signed-off-by: Annie Lee <leeyun@amazon.com> * Update Dashboard.test.js Signed-off-by: Annie Lee <leeyun@amazon.com> * Remove test alert start time Signed-off-by: Annie Lee <leeyun@amazon.com> * Update VisualGraph test and code Signed-off-by: Annie Lee <leeyun@amazon.com> * Update package.json to run unit tests in UTC timezone Signed-off-by: Annie Lee <leeyun@amazon.com> * Update dashboard sample alert start time Signed-off-by: Annie Lee <leeyun@amazon.com> * Update Dashboard test Signed-off-by: Annie Lee <leeyun@amazon.com> * Update Dashboard.test.js.snap Signed-off-by: Annie Lee <leeyun@amazon.com> * Update snapshot file Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Fix the error handling when config index is not found (#173) Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Updated copyright notices and headers. (#168) * Updated CONTRIBUTING.md and NOTICE files. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Updated license headers. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored acknowledge alerts button on Alerts by trigger dashboard page to be a modal experience. (#167) * Refactored Acknowledge button on Alerts by trigger dashboard to display a modal. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> * Updated license headers. Signed-off-by: AWSHurneyt <79280347+AWSHurneyt@users.noreply.github.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Adding a few more basic unit tests (#180) * Add alerting dashboards flyout render unit test Signed-off-by: Annie Lee <leeyun@amazon.com> * Add MonitorTimeFields validation tests Signed-off-by: Annie Lee <leeyun@amazon.com> * Refactor error messages to add period Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Remove node version declaration in package.json (#166) * Remove node version declaration in package.json Signed-off-by: Annie Lee <leeyun@amazon.com> * Remove react-router-dom version declaration Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Update DestinationsService.js (#182) Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Add backport workflow (#176) * Add backport workflow Signed-off-by: Annie Lee <leeyun@amazon.com> * Create delete_backport_branch.yml Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Configure test workflows to run on 1.x branches (#183) Signed-off-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Add 1.3 release notes (#175) (#185) * Create opensearch-alerting-dashboards-plugin.release-notes-1.3.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Update opensearch-alerting-dashboards-plugin.release-notes-1.3.0.0.md Signed-off-by: Annie Lee <leeyun@amazon.com> (cherry picked from commit 98f7904) Co-authored-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Add backport documentation link (#184) (#187) * Create BACKPORT.md Signed-off-by: Annie Lee <leeyun@amazon.com> * Replace documentation with a link Signed-off-by: Annie Lee <leeyun@amazon.com> (cherry picked from commit 0a0c8c5) Co-authored-by: Annie Lee <leeyun@amazon.com> Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Implemented support for cluster metrics monitors (#162) (#189) * Implemented LocalUriInput component, and supporting methods. Implemented unit and integration tests for LocalUriInput. Refactored various other classes and components to support LocalUriInput. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored learn more links to open new tabs. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Implemented default trigger conditions for API supported by the LocalUriInput feature. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored button text and size based on UX reviewer feedback. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Added documentation link for cluster stats API. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Adjusted some wording based on UX reviewer feedback. Implemented modal that displays when changing request type. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored feature assets to use ClusterMetrics naming convention instead of LocalUriInput. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored behavior of cluster metrics clear triggers modal, and implemented tests. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Added periods to error messages. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Removed fixed width for clearTriggersModal. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored the cluster metrics radio card to be a monitor type instead of monitor definition type. Refactored request type selection combobox so it's not clearable, and reworded the clearTriggersModal text and refactored tests accordingly. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Updated copyright headers. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactoring monitorType checking logic to accommodate cluster metrics monitors. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored help text to use lowercase letters. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Implemented a helper method as a temporary solution to some incorrect formatting of cluster metrics monitors returned by the getMonitors API. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored cluster metrics feature to remove support for CAT repositories. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Refactored cluster metrics request types labels to no longer reference CAT. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> * Adjusted the example trigger conditions for two request types. Signed-off-by: AWSHurneyt <hurneyt@amazon.com> Co-authored-by: Annie Lee <71157062+leeyun-amzn@users.noreply.github.com> Co-authored-by: Clay Downs <downsrob@amazon.com> Co-authored-by: Mohammad Qureshi <qreshi@amazon.com> Co-authored-by: Yaliang <49084640+ylwu-amzn@users.noreply.github.com> Co-authored-by: CEHENKLE <henkle@amazon.com> Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: Annie Lee <leeyun@amazon.com> Co-authored-by: Annie Lee <71157062+leeyun-amzn@users.noreply.github.com> Co-authored-by: Clay Downs <downsrob@amazon.com> Co-authored-by: Mohammad Qureshi <qreshi@amazon.com> Co-authored-by: Yaliang <49084640+ylwu-amzn@users.noreply.github.com> Co-authored-by: CEHENKLE <henkle@amazon.com> Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com>

AWSHurneyt requested review from lezzago, qreshi and annie3431 December 10, 2021 00:04

Merge branch 'opensearch-project:main' into issue125-main

2adef8f

AWSHurneyt marked this pull request as ready for review December 20, 2021 18:35

qreshi reviewed Jan 10, 2022

View reviewed changes

public/pages/Dashboard/utils/helpers.test.js Outdated Show resolved Hide resolved

public/components/Flyout/flyouts/components/AlertsDashboardFlyoutComponent.js Outdated Show resolved Hide resolved

Merge branch 'opensearch-project:main' into issue125-main

1e2c855

AWSHurneyt requested a review from a team January 10, 2022 19:21

AWSHurneyt added 2 commits January 10, 2022 11:27

Removed an unused test variable.

d84df6c

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

Removed debug logs.

ea0fb25

Signed-off-by: AWSHurneyt <hurneyt@amazon.com>

annie3431 reviewed Jan 10, 2022

View reviewed changes

cypress/integration/alerts_dashboard_flyout_spec.js Outdated Show resolved Hide resolved