Skip to content

[7.x] [Security Solution][Detections] Handle conflicts on alert status update (#75492)#76804

Merged
madirey merged 2 commits intoelastic:7.xfrom
madirey:backport/7.x/pr-75492
Sep 8, 2020
Merged

[7.x] [Security Solution][Detections] Handle conflicts on alert status update (#75492)#76804
madirey merged 2 commits intoelastic:7.xfrom
madirey:backport/7.x/pr-75492

Conversation

@madirey
Copy link
Copy Markdown
Contributor

@madirey madirey commented Sep 4, 2020

Backports the following commits to 7.x:

…te (elastic#75492)

* Proceed on conflict when updating alert status

* Handle conflicts

* Don't let the user retry

* Tweak error messages

* Fix route

* Update add exception modal

* Reapply changes after fixing conflicts

* Type errors

* types

* Fix remaining conflicts

* Fix tests

* More test fixes

* Simplify onConflict evaluation

* Add callback return types

* Update translation paths

* Add missing import

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@madirey madirey added the backport This PR is a backport of another PR label Sep 4, 2020
@madirey
Copy link
Copy Markdown
Contributor Author

madirey commented Sep 7, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

async chunks size

id value diff baseline
securitySolution 9.9MB +11.7KB 9.9MB

page load bundle size

id value diff baseline
securitySolution 812.9KB +54.0B 812.8KB

History

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

@madirey madirey merged commit 743b7b9 into elastic:7.x Sep 8, 2020
@madirey madirey deleted the backport/7.x/pr-75492 branch September 8, 2020 02:20
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.

3 participants