Skip to content

[8.8] [RAM] Alert search bar only KQL (#155947)#156793

Merged
kibanamachine merged 1 commit intoelastic:8.8from
kibanamachine:backport/8.8/pr-155947
May 5, 2023
Merged

[8.8] [RAM] Alert search bar only KQL (#155947)#156793
kibanamachine merged 1 commit intoelastic:8.8from
kibanamachine:backport/8.8/pr-155947

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.8:

Questions ?

Please refer to the Backport tool documentation

## Summary

Adding the props `showQueryInput` is fixing the layout of of the
filters. We also wanted to make sure that our alert search bar can only
work with KQL because we need to do some kind of validation on the field
used in the search bar, therefore it will be easier to just use
KueryNode for now.

### Before:
<img width="754" alt="Screenshot 2023-04-26 at 1 52 44 PM"
src="https://user-images.githubusercontent.com/189600/234698078-612f03be-3331-41ab-a2d9-80f6cd767043.png">

### After:
<img width="749" alt="image"
src="https://user-images.githubusercontent.com/189600/234697735-abafc15f-5562-42a0-aeb9-638129d70120.png">

---------

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit af7e34a)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label May 5, 2023
@kibanamachine kibanamachine enabled auto-merge (squash) May 5, 2023 07:38
@kibanamachine kibanamachine merged commit d762daa into elastic:8.8 May 5, 2023
@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

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.4MB 1.4MB +132.0B
unifiedSearch 268.4KB 268.6KB +177.0B
total +309.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 17 19 +2
securitySolution 399 402 +3
total +5

Total ESLint disabled count

id before after diff
enterpriseSearch 18 20 +2
securitySolution 479 482 +3
total +5

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

cc @XavierM

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