-
Notifications
You must be signed in to change notification settings - Fork 216
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
Refine Multiple window telemetry #2323
Conversation
69f2498
to
5ea6ecb
Compare
I have confirmed it works as I thought. @keianhzo Let's do the review first, then I will update our Wiki and start the data review then. |
app/src/common/shared/org/mozilla/vrbrowser/telemetry/TelemetryWrapper.java
Outdated
Show resolved
Hide resolved
app/src/common/shared/org/mozilla/vrbrowser/telemetry/TelemetryWrapper.java
Outdated
Show resolved
Hide resolved
5ea6ecb
to
583378f
Compare
@daoshengmu ready to start the data rwview |
Request for data collection review formAll questions are mandatory. You must receive review from a data steward peer on your responses to these questions before shipping new data collection.
|
@harraton Please help do data review, and the wiki has been updated. |
We didn't add new telemetry and just refine existing ones. So. we are going to merge it and please let me know if there is other concerns, we will address it then. |
Fixes #2030.