-
Notifications
You must be signed in to change notification settings - Fork 1.3k
For #9136: Send all adjust properties to glean #9253
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.
|
464ec8a
to
4f10d24
Compare
Codecov Report
@@ Coverage Diff @@
## master #9253 +/- ##
============================================
+ Coverage 19.51% 19.53% +0.01%
Complexity 511 511
============================================
Files 329 329
Lines 13155 13158 +3
Branches 1747 1747
============================================
+ Hits 2567 2570 +3
Misses 10364 10364
Partials 224 224
Continue to review full report at Codecov.
|
4f10d24
to
3daaa1a
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 only
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, documented in metrics.yaml
- Is there a control mechanism that allows the user to turn the data collection on and off? (Note, for data collection not needed for security purposes, Mozilla provides such a control mechanism) Provide details as to the control mechanism available.
Yes, Fenix data controls
- If the request is for permanent data collection, is there someone who will monitor the data over time?
Will expire 9/1
- Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
Type 2, adjust campaign fields based on campaign that user downloaded Fenix from.
- 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)?
Yes, adding clientid to installation ping. This is the same information as the metrics ping, but sent when adjust fields resolve. Omitting the clientid was a bug, which didn't allow tracking retention.
- Is the data collection covered by the existing Firefox privacy notice?
Yes, this includes the clientid but the ping is the same as the metrics ping.
- Does there need to be a check-in in the future to determine whether to renew the data? (Yes/No) (If yes, set a todo reminder or file a bug if appropriate)**
Will be reviewed 9/1/2020
- Does the data collection use a third-party collection tool? If yes, escalate to legal.
No
Pull Request checklist
After merge
To download an APK when reviewing a PR: