Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Alert search bar] Replace the status filter with controls on the observability alerts page #198495

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

maryam-saeidi
Copy link
Member

Closes #197953

Summary

This PR replaces the alert status filter with filter controls.

We need to fix the following items before merging this feature:

  • Solving the permission issue
    • The main issue will be fixed in this PR
    • In the above PR, we remove controls if the user does not have the privilege for alert indices, but we need to figure out how to adjust filter controls to access the data based on Kibana privileges.
  • We should configure the filters to allow the selection of one item for alert status but still show the other options
  • Changing the URL does not update the page filters correctly. Might be related to [ResponseOps][Alerts] Stack alerts page table loads twice #183412.
  • We need to make sure these adjustments work as expected in APM as they use the observability alert search bar.
  • Check if the tags filter can be improved, and if not, whether it makes sense to keep it in its current form.

@maryam-saeidi maryam-saeidi added release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Oct 31, 2024
@maryam-saeidi maryam-saeidi self-assigned this Oct 31, 2024
@elasticmachine
Copy link
Contributor

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Alert search bar] Replace the status filter with controls on the observability alerts page
2 participants