-
Notifications
You must be signed in to change notification settings - Fork 8.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[SLO] Move overview embeddable to new react registry (#181930)
Fixes #179296 ## 🍒 Summary This PR converts the SLO Overview embeddable (Single SLO & Grouped SLOs) to the new React Embeddable framework. There are no UI changes to the current embeddable and the behavior should be the same. I have a video of how the embeddable should work. https://github.com/elastic/kibana/assets/2852703/d37c3f99-3140-4a09-a223-36a1d2242649 ## ✔️ Acceptance criteria - A new `create_overview_panel_action` action is created, which adds the `SLO Overview` option in the Add panel section - The `SLO Overview` menu option is grouped under SLOs menu - Clicking on the single SLO card should open a Flyout with the SLO details - Clicking on the Grouped SLOs should expand the accordion and show the SLOs that belong to this group - The Grouped SLO have the `Edit criteria` option - `Edit criteria` option appears under `More` panel options only for the Grouped SLOs - Kibana screenshot tool should report no timeout error. - Clicking on the Refresh button should reload the embeddable - We should be able to set default dimensions to the embeddable panels cc @nickpeihl - Depends on #182120 --------- Co-authored-by: kibanamachine <[email protected]>
- Loading branch information
1 parent
cd22c67
commit 79b4d5e
Showing
22 changed files
with
452 additions
and
610 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -49,7 +49,8 @@ | |
"kibanaUtils", | ||
"unifiedSearch", | ||
"embeddable", | ||
"ingestPipelines" | ||
"ingestPipelines", | ||
"dashboard" | ||
] | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
16 changes: 16 additions & 0 deletions
16
x-pack/plugins/observability_solution/slo/public/embeddable/slo/common/constants.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License | ||
* 2.0; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
|
||
export const COMMON_SLO_GROUPING = [ | ||
{ | ||
id: 'slos', | ||
getDisplayName: () => 'SLOs', | ||
getIconType: () => { | ||
return 'visGauge'; | ||
}, | ||
}, | ||
]; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
173 changes: 0 additions & 173 deletions
173
...plugins/observability_solution/slo/public/embeddable/slo/overview/slo_embeddable.test.tsx
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.