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

Backport bug fixes and release notes #228

Merged
merged 2 commits into from
Apr 26, 2022

Conversation

lezzago
Copy link
Member

@lezzago lezzago commented Apr 26, 2022

Description

  • Backport bug fixes and release notes

Issues Resolved

N/A

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.

AWSHurneyt and others added 2 commits April 25, 2022 17:00
* Refactored updateMonitor API to no longer require seqNo or primaryTerm.

Signed-off-by: AWSHurneyt <[email protected]>

* Fixed bug that was preventing acknowledge modal from refreshing when acknowledging alerts. Fixed bug that was causing the acknowledge modal to render on the monitor details page instead of immediately acknowledging selected alerts.

Signed-off-by: AWSHurneyt <[email protected]>

* Fixed alerts flyout to render correct pluralization for conditions. Adjusted spacing on action component.

Signed-off-by: AWSHurneyt <[email protected]>

* Implemented logic to hide anomaly detection option when defining a doc level monitor.

Signed-off-by: AWSHurneyt <[email protected]>

* Fixed a bug impacting sending test notifications for doc level monitors.

Signed-off-by: AWSHurneyt <[email protected]>

* Refactored visual editor to no longer support spaces in tags and query names. Fixed a bug that preventing creating a doc level monitor with a notification channel.

Signed-off-by: AWSHurneyt <[email protected]>

* Updated snapshots.

Signed-off-by: AWSHurneyt <[email protected]>
* Add release notes for version 2.0.0-rc1

Signed-off-by: Ashish Agrawal <[email protected]>
@lezzago lezzago requested a review from a team April 26, 2022 00:02
@lezzago lezzago merged commit cf550a0 into opensearch-project:2.0 Apr 26, 2022
AWSHurneyt added a commit to AWSHurneyt/alerting-dashboards-plugin that referenced this pull request Jun 10, 2022
* Document level monitor UX bug fixes (opensearch-project#226)

* Refactored updateMonitor API to no longer require seqNo or primaryTerm.

Signed-off-by: AWSHurneyt <[email protected]>

* Fixed bug that was preventing acknowledge modal from refreshing when acknowledging alerts. Fixed bug that was causing the acknowledge modal to render on the monitor details page instead of immediately acknowledging selected alerts.

Signed-off-by: AWSHurneyt <[email protected]>

* Fixed alerts flyout to render correct pluralization for conditions. Adjusted spacing on action component.

Signed-off-by: AWSHurneyt <[email protected]>

* Implemented logic to hide anomaly detection option when defining a doc level monitor.

Signed-off-by: AWSHurneyt <[email protected]>

* Fixed a bug impacting sending test notifications for doc level monitors.

Signed-off-by: AWSHurneyt <[email protected]>

* Refactored visual editor to no longer support spaces in tags and query names. Fixed a bug that preventing creating a doc level monitor with a notification channel.

Signed-off-by: AWSHurneyt <[email protected]>

* Updated snapshots.

Signed-off-by: AWSHurneyt <[email protected]>

* Add release notes for version 2.0.0-rc1 (opensearch-project#227)

* Add release notes for version 2.0.0-rc1

Signed-off-by: Ashish Agrawal <[email protected]>

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

Successfully merging this pull request may close these issues.

3 participants