Skip to content

Conversation

@dmitri-saricev-3pillargloball
Copy link

@dmitri-saricev-3pillargloball dmitri-saricev-3pillargloball commented Nov 24, 2025

…install

What is this feature?
https://perconadev.atlassian.net/browse/PMM-14080

[Add a brief description of what the feature or update does.]

Why do we need this feature?

[Add a description of the problem the feature is trying to solve.]

Who is this feature for?

[Add information on what kind of user the feature is for.]

Which issue(s) does this PR fix?:
Screenshot 2025-11-24 at 10 33 12
Screenshot 2025-11-24 at 10 32 41

Screen.Recording.2025-11-24.at.11.14.49.mov

FB:
Percona-Lab/pmm-submodules#4126

https://pmm.cd.percona.com/blue/organizations/jenkins/pmm3-submodules/detail/pmm3-submodules/1956/pipeline/

Fixes #

Special notes for your reviewer:

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.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses the lack of guidance on the Fired Alerts page after installation by adding an informative empty state component. When no alerts are detected, users are now presented with helpful links and documentation to help them set up alerting.

  • Added a new AlertsEmptyState component with guidance on setting up alerts
  • Integrated the empty state component into the Alerts table view
  • Provided direct links to Alert rule templates, Alert rules configuration, and Percona documentation

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
AlertsEmptyState.tsx New component that displays guidance with links to alert templates, rules configuration, and documentation when no alerts are present
Alerts.tsx Updated to use the new AlertsEmptyState component instead of a simple icon and text message

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants