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.

Sign up for GitHub

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

Jump to bottom

[Actionable Observability] Use new Alert Summary API in Alert Summary Widget #147762

Closed
Tracked by #137196
maryam-saeidi opened this issue Dec 19, 2022 · 1 comment · Fixed by #147784
Closed
Tracked by #137196

[Actionable Observability] Use new Alert Summary API in Alert Summary Widget #147762

maryam-saeidi opened this issue Dec 19, 2022 · 1 comment · Fixed by #147784
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.7.0

Comments

@maryam-saeidi
Copy link
Member

📝 Summary

In this PR, we have a new API to get the Alert Summary information for related charts. In the first step, we need to use this API to get the number of active and recovered alerts.

We also need to refactor the AlertSummaryWidget component to rely only on featureIds and remove the extra rule prop so that we can use this component outside the rule details page.

✅ Acceptance Criteria

  • Use the new Alert Summary API in AlertSummaryWidget
  • Remove extra rule prop from AlertSummaryWidget component
@maryam-saeidi maryam-saeidi added Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.7.0 labels Dec 19, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/actionable-observability (Team: Actionable Observability)

This was referenced Dec 19, 2022
Closed
Merged
maryam-saeidi added a commit that referenced this issue Dec 21, 2022
@maryam-saeidi
[AO] Use new API in Alert Summary Widget and remove rule prop (#147784)
6522ea0 
Resolves #147762 

## 📝 Summary 

Use new API in Alert Summary Widget and remove rule prop

## 🧪 How to test

No functionality has changed from the user's perspective, you should see
the Alert Summary Widget work as before on the rule details page.

![image](https://user-images.githubusercontent.com/12370520/208632625-0d3935fd-661a-45ce-bdb1-5a87e2da87cd.png)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.7.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[AO] Use new API in Alert Summary Widget and remove rule prop maryam-saeidi/kibana
2 participants
@maryam-saeidi @elasticmachine