-
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
[Telemetry] Centralized opt in service #48505
Conversation
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
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.
Sass changes are minimal. Didn't look beyond the code.
@elasticmachine merge upstream |
💔 Build Failed |
@Bamieh What's the status here? My main concern is getting those uiExport hacks out of legacy and into the New Platform. |
@joshdover This has been pushed back due to the 7.5.0 release urgent features. With all the changes introduced to the telemetry repo in the meantime; I believe i'll be closing this PR and cherrypicking files from the branch. I have two PRs migrating things into NP for this week ( |
Summary
$injector
and angularProvider
in telemetry to further ease moving to NP.z-index
.fetchTelemetry
code.Closes #45428
Remaining:
telemetry.enabled
is set tofalse