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 2.7] [Backport 2.x] 2.7 manual backports. #533

Merged
merged 1 commit into from
Apr 18, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport d0b1702 from #532

* Minor bug fixes (#505)

* [BUG] Edit details Save button should be blue #499

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [BUG] Detector details has some bad text handling #496

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [BUG] Detector details has some bad text handling #496

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [BUG] Detector details has some bad text handling #496

Signed-off-by: Jovan Cvetkovic <[email protected]>

---------

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

* [BUG] Fixes few minor UX bugs (#525)

* [BUG] No space between the detector details and the rule panel #522
[BUG] A rule flyout without references have an empty link #521
[FEATURE] Update header size to use euiTitle--small #520

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [BUG] No space between the detector details and the rule panel #522
[BUG] A rule flyout without references have an empty link #521
[FEATURE] Update header size to use euiTitle--small #520

Signed-off-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Feature] deleting detectors should delete all related dashboards (including index patterns and visualisations) (#515)

* Update detector details component #504

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [FEATURE] Deleting detectors should delete all related dashboards (including index-patterns and visualisations) #509

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [FEATURE] Deleting detectors should delete all related dashboards (including index-patterns and visualisations) #509

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [FEATURE] Deleting detectors should delete all related dashboards (including index-patterns and visualisations) #509

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [FEATURE] Deleting detectors should delete all related dashboards (including index-patterns and visualisations) #509

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [FEATURE] Deleting detectors should delete all related dashboards (including index-patterns and visualisations) #509

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [FEATURE] Deleting detectors should delete all related dashboards (including index-patterns and visualisations) #509

Signed-off-by: Jovan Cvetkovic <[email protected]>

* [FEATURE] Deleting detectors should delete all related dashboards (including index-patterns and visualisations) #509

Signed-off-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Resolved merge conflicts, generated updated snapshot files, resolved discrepancies between main and 2.x branch.

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

* Resolved merge conflicts, generated updated snapshot files, resolved discrepancies between main and 2.x branch.

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

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
(cherry picked from commit d0b1702)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team April 18, 2023 21:11
@codecov-commenter
Copy link

Codecov Report

Merging #533 (34f8fb5) into 2.7 (b483cd1) will increase coverage by 0.47%.
The diff coverage is n/a.

📣 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             @@
##              2.7     #533      +/-   ##
==========================================
+ Coverage   35.05%   35.53%   +0.47%     
==========================================
  Files         122      121       -1     
  Lines        3500     3487      -13     
  Branches      612      609       -3     
==========================================
+ Hits         1227     1239      +12     
+ Misses       2138     2108      -30     
- Partials      135      140       +5     

see 14 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.

@AWSHurneyt AWSHurneyt merged commit 621ff95 into 2.7 Apr 18, 2023
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