-
Notifications
You must be signed in to change notification settings - Fork 1.3k
For #6396 - Added metrics for history/bookmarks account sync #6601
For #6396 - Added metrics for history/bookmarks account sync #6601
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.
Until 03/01/2020
|
a53aa7c
to
77377e9
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)
Instructions: Data Stewards will review a request for data collection and endorse responses to each question. If the request does not provide answers to questions, reviewers give an r- and point to the questions that can’t be answered.
-
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
Codecov Report
@@ Coverage Diff @@
## master #6601 +/- ##
============================================
- Coverage 17.3% 17.28% -0.02%
Complexity 391 391
============================================
Files 281 281
Lines 11225 11235 +10
Branches 1546 1546
============================================
Hits 1942 1942
- Misses 9128 9138 +10
Partials 155 155
Continue to review full report at Codecov.
|
77377e9
to
e439197
Compare
…s account sync toggles
e439197
to
be94e69
Compare
Pull Request checklist
After merge
To download an APK when reviewing a PR: