-
Notifications
You must be signed in to change notification settings - Fork 1.3k
For #7289: Adds telemetry for selecting dark theme #7968
Conversation
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.
|
Codecov Report
@@ Coverage Diff @@
## master #7968 +/- ##
============================================
+ Coverage 19.15% 19.18% +0.02%
+ Complexity 454 452 -2
============================================
Files 306 305 -1
Lines 11902 11853 -49
Branches 1603 1596 -7
============================================
- Hits 2280 2274 -6
+ Misses 9431 9388 -43
Partials 191 191
Continue to review full report at Codecov.
|
2af3ce6
to
01cc65f
Compare
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.
Data Review Form (to be filled by Data Stewards)
-
Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?
Yes, metrics.yaml and metrics.md -
Is there a control mechanism that allows the user to turn the data collection on and off?
Yes, under data controls -
If the request is for permanent data collection, is there someone who will monitor the data over time?
Has expiry -
Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
Type 2 -
Is the data collection request for default-on or default-off?
Default on -
Does the instrumentation include the addition of any new identifiers (whether anonymous or otherwise; e.g., username, random IDs, etc. See the appendix for more details)?
No -
Is the data collection covered by the existing Firefox privacy notice?
Yes -
Does there need to be a check-in in the future to determine whether to renew the data?
Will check-in at expiry -
Does the data collection use a third-party collection tool?
No
Pull Request checklist
After merge
To download an APK when reviewing a PR: