-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Put APM links into header action menu #82292
Conversation
retest |
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 - just one question around useServiceName
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@sqren @shahzad31 I updated this to use the reverse portal. Please re-review. |
@elasticmachine merge upstream |
💚 Build SucceededMetrics [docs]Module Count
Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
APM
react-reverse-portal
to mount it there.appMountParameters
(which includeshistory
andsetHeaderActionMenu
) in the APM and UX plugin contexts.isServiceMapEnabled
check from the home tabs and make it a constant instead of a function.Before
After
Observability
setHelpExtension
menuBefore
After
References #81954.