Skip to content

[7.x] Rename alert status OK to Recovered and fix some UX issues around disabling a rule while being in an error state (#98135)#100063

Merged
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-98135
May 13, 2021
Merged

[7.x] Rename alert status OK to Recovered and fix some UX issues around disabling a rule while being in an error state (#98135)#100063
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-98135

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…abling a rule while being in an error state (elastic#98135)

* Fix UX when alert is disabled and in an error state

* Reset executionStatus to pending after enabling an alert

* Renames alert instance status OK to Recovered

* Fix end to end test

* Update doc screenshot

* Fix confusing test name

* Remove flakiness in integration test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine
Copy link
Copy Markdown
Contributor Author

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
triggersActionsUi 1.5MB 1.5MB +93.0B
Unknown metric groups

References to deprecated APIs

id before after diff
fleet 4 2 -2
globalSearch 4 2 -2
licensing 18 15 -3
total -7

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

cc @mikecote

@kibanamachine kibanamachine merged commit 02efbb6 into elastic:7.x May 13, 2021
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