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

Add release notes for version 2.0.0-rc1 #227

Merged
merged 2 commits into from
Apr 25, 2022

Conversation

lezzago
Copy link
Member

@lezzago lezzago commented Apr 25, 2022

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

Description

  • Add release notes for version 2.0.0-rc1

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.

@lezzago lezzago requested a review from a team April 25, 2022 23:14
@AWSHurneyt
Copy link
Collaborator

Should the notes include #226?

### Enhancements
* Implemented UX support for configuring doc level monitors. ([#218](https://github.com/opensearch-project/alerting-dashboards-plugin/pull/218))
* Integrate Alerting Dashboards with Notifications Plugin ([#220](https://github.com/opensearch-project/alerting-dashboards-plugin/pull/220))
* Remove disabled buttons and update Destination flows to reflect read-only state ([#221](https://github.com/opensearch-project/alerting-dashboards-plugin/pull/221))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wondering if this should go as part of Refactor?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I meant for #221 only

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@getsaurabh02
Copy link
Member

@AWSHurneyt Isn't #226 already covered as part of (#218 - Implemented UX support for configuring doc level monitors.) since this is first time doc level alerting is being released?

Signed-off-by: Ashish Agrawal <[email protected]>
@lezzago
Copy link
Member Author

lezzago commented Apr 25, 2022

I agree with @AWSHurneyt since based on existing standard, we would still want to include #226 under bug fixes

@lezzago lezzago merged commit a082826 into opensearch-project:main Apr 25, 2022
lezzago added a commit to lezzago/alerting-dashboards-plugin-1 that referenced this pull request Apr 26, 2022
* Add release notes for version 2.0.0-rc1

Signed-off-by: Ashish Agrawal <[email protected]>
lezzago added a commit that referenced this pull request Apr 26, 2022
* Document level monitor UX bug fixes (#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 (#227)

* Add release notes for version 2.0.0-rc1

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

Co-authored-by: AWSHurneyt <[email protected]>
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.

4 participants