Skip to content

[8.0] [8.0][RAC] 117686 replace alert workflow status in alerts view (#118723)#120198

Merged
fkanout merged 1 commit intoelastic:8.0from
fkanout:backport/8.0/pr-118723
Dec 2, 2021
Merged

[8.0] [8.0][RAC] 117686 replace alert workflow status in alerts view (#118723)#120198
fkanout merged 1 commit intoelastic:8.0from
fkanout:backport/8.0/pr-118723

Conversation

@fkanout
Copy link
Copy Markdown
Contributor

@fkanout fkanout commented Dec 2, 2021

Backports the following commits to 8.0:

…ic#118723)

* Add AlertStatus types

* Add alert status filter component

* Remove Filter in action from the t grid table

* Update group buttons to applied Alert status filter instead of Workflow status

* Keep the Alert status button in sync when typing and first page load

* Fix data test object name and translation keys label

* Add possibility to hide the bulk actions

* Update how hide the bulk actions

* Fix showCheckboxes hardcoded "true". Instead use the leadingControlColumns props

* Hide the leading checkboxes  in the T Grid with the bulk actions

* Update showCheckboxes to false

* Fix test as the leading checkboxes are hidden

* Update tests

* Get back disabledCellActions as it's required by T Grid

* Update tests to skip test related to Workflow action buttons

* Skip workflow tests

* Revert fix showCheckboxes

* Remove unused imports

* Revert the o11y tests as the checkBoxes fix is reverted

* Reactive the tests effected by checkBoxes

* Skip alert workflow status

* [Code review] use predefined types

* Remove unused prop

* Use the alert-data index name in the RegEx

* Detect * in KQL as "show al"l alert filter

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@fkanout fkanout added the backport This PR is a backport of another PR label Dec 2, 2021
@fkanout fkanout enabled auto-merge (squash) December 2, 2021 09:58
@fkanout fkanout merged commit 7b9763f into elastic:8.0 Dec 2, 2021
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
observability 339 337 -2

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
observability 369.9KB 368.7KB -1.1KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
observability 79.9KB 79.7KB -139.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants