Skip to content

[7.x] [Security Solutions][Detection Engine] Creates an autocomplete package and moves duplicate code between lists and security_solution there (#105382)#106592

Closed
kibanamachine wants to merge 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-105382
Closed

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…e and moves duplicate code between lists and security_solution there (elastic#105382)

## Summary

Creates an autocomplete package from `lists` and removes duplicate code between `lists` and `security_solutions`
* Consolidates different PR's where we were changing different parts of autocomplete in different ways.
* Existing Cypress tests should cover any mistakes hopefully

Manual Testing:
* Ensure this bug does not crop up again elastic#87004
* Make sure that the exception list autocomplete looks alright

### Checklist

- [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
@kibanamachine
Copy link
Copy Markdown
Contributor Author

kibanamachine commented Jul 22, 2021

💔 Build Failed

Failed CI Steps

Metrics [docs]

Unknown metric groups

References to deprecated APIs

id before after diff
lists 233 94 -139
securitySolution 359 272 -87
total -226

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

cc @FrankHassanabad

@FrankHassanabad
Copy link
Copy Markdown
Contributor

From the failure and weird conflict, closing in favor of manually running the backport and fixing conflicts here:
#106612

auto-merge was automatically disabled July 22, 2021 22:01

Pull request was closed

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