-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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.
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
[Observability] Passes theme$
to toMountPoint
#120968
Conversation
Pinging @elastic/uptime (Team:uptime) |
Pinging @elastic/apm-ui (Team:apm) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apm changes lgtm
theme$
to toRenderMount
theme$
to toMountPoint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Infra changes look good.
Pretty sure this covers #120713 so I'll close that in favor of this one. Thanks! |
Ah, looks like this still might be missing the toMountPoint call sites mentioned in #120713 (review) TBH I'm still pretty confused by all this, but I guess let's land this PR then we can do a follow up to figure out what to do about getting the theme down to the toasts as well. |
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for uptime plugin.
💔 Backport failedThe backport operation could not be completed due to the following error: The backport PRs will be merged automatically after passing CI. To backport manually run: |
@claudiopro do we need a backport here? I'm thinking no. |
I didn't backport the corresponding PR for the |
… toast notifications (#120713) This PR originally added the KibanaThemeProvider to stack monitoring as well, but #120968 was opened to do the same thing. Instead this change focuses on updating the toMountPoint calls found in stack monitoring toast notifications to ensure they're either handled by simple i18n calls or appropriately pass a theme$ argument when called.
@matschaffer @weltenwort That's my understanding too, but happy to backport if needed. |
* Treats toMountPoint - part II * Adds theme$ to exploratory view config * Uses theme$ from useExploratoryView hook * Uses theme$ in HeaderActionMenuContext * Better props for MonitoringApp
… toast notifications (elastic#120713) This PR originally added the KibanaThemeProvider to stack monitoring as well, but elastic#120968 was opened to do the same thing. Instead this change focuses on updating the toMountPoint calls found in stack monitoring toast notifications to ensure they're either handled by simple i18n calls or appropriately pass a theme$ argument when called.
* Treats toMountPoint - part II * Adds theme$ to exploratory view config * Uses theme$ from useExploratoryView hook * Uses theme$ in HeaderActionMenuContext * Better props for MonitoringApp
… toast notifications (elastic#120713) This PR originally added the KibanaThemeProvider to stack monitoring as well, but elastic#120968 was opened to do the same thing. Instead this change focuses on updating the toMountPoint calls found in stack monitoring toast notifications to ensure they're either handled by simple i18n calls or appropriately pass a theme$ argument when called.
Summary
This PR adds options containing the value of
theme$
as second argument oftoMountPoint
, passing it down fromAppMountParams
. It is the continuation of work initiated with #120400.Closes #120896
Rel #118906
Checklist