Create opensearch-dashboards-reports.release-notes-1.2.0.0.md #219
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
| ### **Version 1.2.0.0 Release Notes** | ||
|
There was a problem hiding this comment. Is there an update in the 1.2 release notes to make the section titles in bold? Our past release notes don't have this style There was a problem hiding this comment. I followed the Version 1.1.0.0 Release Notes title style which i thought was in bold so went with it. I'll change it |
||
| Compatible with OpenSearch and OpenSearch Dashboards Version 1.2.0 | ||
|
|
||
| ### **Bug Fixes** | ||
| * Fix quoting and url-encoding ([#153](https://github.com/opensearch-project/dashboards-reports/pull/153)) | ||
| * Remove hard coded localhost when calling API ([#172](https://github.com/opensearch-project/dashboards-reports/pull/172)) | ||
| * rename plugin helper config file name to consistent with OSD ([#180](https://github.com/opensearch-project/dashboards-reports/pull/180)) | ||
|
There was a problem hiding this comment. this is |
||
| * Support range filters for csv reports ([#185](https://github.com/opensearch-project/dashboards-reports/pull/185)) | ||
| * Fix Report Creation after Report Definition Creation ([#196](https://github.com/opensearch-project/dashboards-reports/pull/196)) | ||
| * fix csv missing fields issue and empty csv on _source fields ([#206](https://github.com/opensearch-project/dashboards-reports/pull/206)) | ||
|
|
||
|
|
||
| ### **Infrastructure** | ||
| * Taking RBAC settings from Alerting plugin default to false ([#165](https://github.com/opensearch-project/dashboards-reports/pull/165)) | ||
| * Integrate notifications backend ([#129](https://github.com/opensearch-project/dashboards-reports/pull/129)) | ||
|
There was a problem hiding this comment. this is also part of adding notifications (which was then removed), could you also put in enhancements like others |
||
| * Build email message from template with reports links ([#184](https://github.com/opensearch-project/dashboards-reports/pull/184)) | ||
| * Use advanced settings for date format in csv reports ([#186](https://github.com/opensearch-project/dashboards-reports/pull/186)) | ||
| * Remove Notifications from Create/Edit ([#212](https://github.com/opensearch-project/dashboards-reports/pull/212)) | ||
| * Remove calling notifications in reports scheduler ([#211](https://github.com/opensearch-project/dashboards-reports/pull/211)) | ||
|
There was a problem hiding this comment. i think you can put them all in There was a problem hiding this comment. ohh got it. I'll change accordingly |
||
|
|
||
|
|
||
| ### **Enhancements** | ||
| * Add metrics for notifications ([#173](https://github.com/opensearch-project/dashboards-reports/pull/173)) | ||
| * Add logic to build report detail page link and send as part of message for non-email channels ([#182](https://github.com/opensearch-project/dashboards-reports/pull/182)) | ||
| * Forward all cookies while using headless Chromium ([#194](https://github.com/opensearch-project/dashboards-reports/pull/194)) | ||
| * Catch Notifications Errors on Details Pages ([#197](https://github.com/opensearch-project/dashboards-reports/pull/197)) | ||
| * Remove notifications integration from Details pages ([#210](https://github.com/opensearch-project/dashboards-reports/pull/210)) | ||
|
There was a problem hiding this comment. How come some of the PRs for removing notifications are in Infrastructure and others are in Enhancements? I feel they should all be in the same section There was a problem hiding this comment. Shall i put all PRs for removing notifications under Infrastructure ? There was a problem hiding this comment. I would say it should be under Infrastructure, thoughts @joshuali925 ? There was a problem hiding this comment. we use infra to categorize CI/CD, pipeline, UT/IT changes. Not very sure what goes for removing notifications, maybe |
||
| * refactor logic for creating DSL from saved object using `buildOpensearchQuery()` ([#213](https://github.com/opensearch-project/dashboards-reports/pull/213)) | ||
| * Use advanced settings for csv separator and visual report timezone ([#209](https://github.com/opensearch-project/dashboards-reports/pull/209)) | ||
|
|
||
| ### **Documentation** | ||
| * Update validation for observability notebooks integration ([#174](https://github.com/opensearch-project/dashboards-reports/pull/174)) | ||
| * Add dco and release drafter workflows ([#217](https://github.com/opensearch-project/dashboards-reports/pull/217)) | ||
| * update README notification section ([#216](https://github.com/opensearch-project/dashboards-reports/pull/216)) | ||
|
|
||
| ### **Maintenance** | ||
| * Bump to version 1.2 ([#203](https://github.com/opensearch-project/dashboards-reports/pull/203)) | ||
| * Bump tmpl from 1.0.4 to 1.0.5 in /dashboards-reports ([#164](https://github.com/opensearch-project/dashboards-reports/pull/164)) | ||
| * Revert backend paths to opendistro ([#218](https://github.com/opensearch-project/dashboards-reports/pull/218)) | ||
|
There was a problem hiding this comment. just the PR 218 right? |
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you remove the
**? I don't think we use thisThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah i'll remove it