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

Alerting: Render folder selector in options for Alert List Panel only when having Grafana datasource #70816

Conversation

soniaAguilarPeiron
Copy link
Member

What is this feature?

This PR modifies the alert list panel to render the folder selector in the options section only when either no datasource is selected or "Grafana" is chosen as the datasource.

Additional this PR makes a small change to the order in which the folder and datasource are displayed.It swaps their positions to improve the user experience.

Why do we need this feature?

This change improves the user experience by providing the folder selector specifically for these scenarios, ensuring relevant options are displayed.

Who is this feature for?

All users

Special notes for your reviewer:

folder-in-alert-panel.mp4

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

…atasource is selected or when Grafana is selected as the datasource
@soniaAguilarPeiron soniaAguilarPeiron added this to the 10.1.x milestone Jun 28, 2023
@soniaAguilarPeiron soniaAguilarPeiron requested a review from a team June 28, 2023 10:17
@soniaAguilarPeiron soniaAguilarPeiron self-assigned this Jun 28, 2023
@soniaAguilarPeiron soniaAguilarPeiron changed the title Alerting: Render folder selector in options for Alert List Panel only when no datasource is selected or when Grafana is selected as the datasource Alerting: Render folder selector in options for Alert List Panel only when having Grafana datasource Jun 28, 2023
Copy link
Contributor

@VikaCep VikaCep left a comment

Choose a reason for hiding this comment

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

:shipit:

@soniaAguilarPeiron soniaAguilarPeiron merged commit 519e840 into main Jun 28, 2023
20 checks passed
@soniaAguilarPeiron soniaAguilarPeiron deleted the alerting/remmove-folder-from-alert-list-panel-if-no-grafana branch June 28, 2023 13:38
harisrozajac pushed a commit that referenced this pull request Jun 29, 2023
… when having Grafana datasource (#70816)

Render folder selector in options for Alert List Panel only when no datasource is selected or when Grafana is selected as the datasource
harisrozajac pushed a commit that referenced this pull request Jun 30, 2023
… when having Grafana datasource (#70816)

Render folder selector in options for Alert List Panel only when no datasource is selected or when Grafana is selected as the datasource
@ricky-undeadcoders ricky-undeadcoders modified the milestones: 10.1.x, 10.1.0 Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants