Skip to content

[AO] Add eui error boundary to observability pages#152786

Merged
maryam-saeidi merged 3 commits intoelastic:mainfrom
maryam-saeidi:152785-add-error-boundary
Mar 8, 2023
Merged

[AO] Add eui error boundary to observability pages#152786
maryam-saeidi merged 3 commits intoelastic:mainfrom
maryam-saeidi:152785-add-error-boundary

Conversation

@maryam-saeidi
Copy link
Copy Markdown
Member

Closes #152785

Summary

Inspired by #65456

Before After
image image

@maryam-saeidi maryam-saeidi added release_note:skip Skip the PR/issue when compiling release notes 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
@maryam-saeidi maryam-saeidi requested a review from a team March 7, 2023 09:23
@maryam-saeidi maryam-saeidi self-assigned this Mar 7, 2023
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
observability 1.1MB 1.1MB +86.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
securitySolution 428 430 +2

Total ESLint disabled count

id before after diff
securitySolution 505 507 +2

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @maryam-saeidi

Copy link
Copy Markdown
Contributor

@benakansara benakansara left a comment

Choose a reason for hiding this comment

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

LGTM. Nice improvement! 👏

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

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes 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

Development

Successfully merging this pull request may close these issues.

[AO] Add eui error boundary to observability plugin

5 participants