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

[AO] Add eui error boundary to observability plugin #152785

Closed
maryam-saeidi opened this issue Mar 7, 2023 · 1 comment · Fixed by #152786
Closed

[AO] Add eui error boundary to observability plugin #152785

maryam-saeidi opened this issue Mar 7, 2023 · 1 comment · Fixed by #152786
Assignees
@maryam-saeidi
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.8.0

Comments

@maryam-saeidi
Copy link
Member

📝 Summary

Introduce eui error boundary for observability pages.

✅ Acceptance Criteria

  • Introduce eui error boundary for observability pages and keep the navigation even when the page's content has an error while rendering.
@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.8.0 labels Mar 7, 2023
@elasticmachine
Copy link
Contributor

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

@maryam-saeidi maryam-saeidi self-assigned this Mar 7, 2023
@maryam-saeidi maryam-saeidi mentioned this issue Mar 7, 2023
Merged
maryam-saeidi added a commit that referenced this issue Mar 8, 2023
@maryam-saeidi
[AO] Add eui error boundary to observability pages (#152786)
662689b 
Closes #152785 

## Summary

Inspired by #65456

|Before|After|
|---|---|

|![image](https://user-images.githubusercontent.com/12370520/223378684-3d7927cf-b1f3-4ebf-a4b8-9deab9bcc23b.png)|![image](https://user-images.githubusercontent.com/12370520/223378731-4c4861f7-afaf-42f8-bc47-9c4a0d7454ec.png)|
bmorelli25 pushed a commit to bmorelli25/kibana that referenced this issue Mar 10, 2023
@maryam-saeidi @bmorelli25
[AO] Add eui error boundary to observability pages (elastic#152786)
a5d88e9 
Closes elastic#152785 

## Summary

Inspired by elastic#65456

|Before|After|
|---|---|

|![image](https://user-images.githubusercontent.com/12370520/223378684-3d7927cf-b1f3-4ebf-a4b8-9deab9bcc23b.png)|![image](https://user-images.githubusercontent.com/12370520/223378731-4c4861f7-afaf-42f8-bc47-9c4a0d7454ec.png)|
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maryam-saeidi maryam-saeidi

Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.8.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[AO] Add eui error boundary to observability pages maryam-saeidi/kibana
2 participants
@maryam-saeidi @elasticmachine