-
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
[Infra UI] Wrap infra content in RedirectAppLinks #177747
[Infra UI] Wrap infra content in RedirectAppLinks #177747
Conversation
Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services) |
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
…ap-infra-app-in-redirectapplinks
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... thanks for improving it
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!
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
Closes #177056
Summary
This PR adds
RedirectAppLinks
wrapper to the infra content and removes it from the links using it. The links that are changed:Show all
: APM and Alerts linksOpen in Logs
Start Trial
linkTesting
test_links_without_alerts_services.mov
test_links_with_services_and_alerts.mov
test_single_service_link.mov