Skip to content

[8.x] [ML] AIOps Hiding saved query controls (#210556)#210928

Merged
jgowdyelastic merged 1 commit intoelastic:8.xfrom
jgowdyelastic:backport/8.x/pr-210556
Feb 13, 2025
Merged

[8.x] [ML] AIOps Hiding saved query controls (#210556)#210928
jgowdyelastic merged 1 commit intoelastic:8.xfrom
jgowdyelastic:backport/8.x/pr-210556

Conversation

@jgowdyelastic
Copy link
Copy Markdown
Member

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

The AIOps pages in ML make use of the shared query bar component, but do
not support saved queries.
By having the load and save query options in the menu, it can give the
impression that these options are generally available but for some
reason cannot be used.
This PR adds a new optional flag which allows the load and save options
to be hidden from the menu.
The flag defaults to always showing the options.

Fixes elastic#209152

**Before**

![image](https://github.com/user-attachments/assets/a26a73d0-63b4-47c1-bfa2-8579b08dfe24)

**After**

![image](https://github.com/user-attachments/assets/4fe6937e-3c7d-4d15-98da-fdf472fc21ee)

(cherry picked from commit 36c764e)

# Conflicts:
#	src/platform/plugins/shared/unified_search/public/query_string_input/query_bar_menu_panels.tsx
@jgowdyelastic jgowdyelastic added the backport This PR is a backport of another PR label Feb 12, 2025
@jgowdyelastic jgowdyelastic enabled auto-merge (squash) February 12, 2025 20:06
Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jgowdyelastic jgowdyelastic merged commit 1f2cf4f into elastic:8.x Feb 13, 2025
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

2 participants