Skip to content

Initial Core Telemetry adoption. #5002

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.

Sign up for GitHub

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

Jump to bottom
Closed
tarigo wants to merge 13 commits into from
Closed

Initial Core Telemetry adoption. #5002

tarigo wants to merge 13 commits into from

Conversation

@tarigo
Copy link

@tarigo tarigo commented Oct 19, 2021 •
edited
Loading

This is an experimental PR with Core Telemetry integrated in addition to MME. The intention behind is to have them both in the same time in order to check/compare data collected on the backend.

@tarigo tarigo requested a review from a team as a code owner October 19, 2021 11:46
@zugaldia zugaldia added Core Work related to core navigation and integrations. upstream Issue has dependency on another component. v2.x Related to version 2.x. ⚠️ DO NOT MERGE PR should not be merged! labels Oct 19, 2021
@zugaldia zugaldia added this to the v2.1 milestone Oct 19, 2021
@tarigo tarigo marked this pull request as draft October 19, 2021 17:43
@tarigo tarigo force-pushed the tarigo/core_telemetry_adoption branch 3 times, most recently from 0bb7474 to 2f11361 Compare October 21, 2021 09:38
@tarigo tarigo changed the title Tarigo/core telemetry adoption Initial Core Telemetry adoption. Oct 21, 2021
@tarigo tarigo added the skip changelog Should not be added into version changelog. label Oct 21, 2021
@tarigo tarigo force-pushed the tarigo/core_telemetry_adoption branch 3 times, most recently from 44ff9dc to 69ffbd2 Compare October 21, 2021 14:39
@tarigo tarigo self-assigned this Oct 21, 2021
@tarigo tarigo force-pushed the tarigo/core_telemetry_adoption branch 3 times, most recently from b3a52f8 to 2e562c6 Compare October 25, 2021 14:38
@tarigo tarigo force-pushed the tarigo/core_telemetry_adoption branch 3 times, most recently from c7b55c9 to 3b6e93a Compare October 29, 2021 10:25
Łukasz Paczos and others added 2 commits December 3, 2021 14:59
@Zayankovsky
add notes for removed v2.1.0 (#5217)
b2b7c66 
* add notes for removed v2.1.0

* typo

Co-authored-by: Vitaly Zayankovsky <[email protected]>

Co-authored-by: Vitaly Zayankovsky <[email protected]>
@Zayankovsky
improve notification on android 12 (#5221)
66c48ec 
Co-authored-by: Vitaly Zayankovsky <[email protected]>
@nagineni nagineni force-pushed the tarigo/core_telemetry_adoption branch from d55ec4a to 2faa9ba Compare December 8, 2021 15:55
@nagineni nagineni force-pushed the tarigo/core_telemetry_adoption branch from 2faa9ba to 0e1c5a5 Compare December 14, 2021 16:01
@Guardiola31337
Copy link
Contributor

Guardiola31337 commented Oct 6, 2022

Hey @tarigo @nagineni now that #6423 landed are we good to close here?

cc @tatiana-yan @RingerJK

@tarigo
Copy link
Author

tarigo commented Oct 7, 2022

Hey @tarigo @nagineni now that #6423 landed are we good to close here?

cc @tatiana-yan @RingerJK

Yes, sure.

@tarigo tarigo closed this Oct 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Guardiola31337 Guardiola31337 Awaiting requested review from Guardiola31337

@RingerJK RingerJK Awaiting requested review from RingerJK

Assignees

@tarigo tarigo

Labels

Core Work related to core navigation and integrations. ⚠️ DO NOT MERGE PR should not be merged! skip changelog Should not be added into version changelog. upstream Issue has dependency on another component. v2.x Related to version 2.x.

Projects

None yet

Milestone

v2.1

Development

Successfully merging this pull request may close these issues.

5 participants

@tarigo @Guardiola31337 @zugaldia @kmadsen @abhishek1508