Skip to content

[8.19] [Obs AI Assistant] replace EuiErrorBoundary with KibanaErrorBoundary (#229710)#229856

Merged
kibanamachine merged 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-229710
Jul 29, 2025
Merged

[8.19] [Obs AI Assistant] replace EuiErrorBoundary with KibanaErrorBoundary (#229710)#229856
kibanamachine merged 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-229710

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…lastic#229710)

Replaces `EuiErrorBoundary` with using `core.rendering.addContext()`.

https://github.com/elastic/kibana/blob/main/src/core/packages/rendering/browser/README.mdx

I tested this by throwing an error in the feedback buttons and the AI
Assistant button itself and confirmed they are caught by the
`KibanaErrorBoundary`.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit d73f9ee)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jul 29, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) July 29, 2025 17:29
@kibanamachine kibanamachine merged commit 1a83bd5 into elastic:8.19 Jul 29, 2025
11 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
observabilityAIAssistantApp 16.1KB 16.1KB -20.0B

cc @neptunian

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

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants