Conversation
Codecov Report
@@ Coverage Diff @@
## master #14468 +/- ##
============================================
- Coverage 30.11% 29.86% -0.25%
Complexity 1167 1167
============================================
Files 444 445 +1
Lines 17920 18074 +154
Branches 2333 2349 +16
============================================
+ Hits 5396 5398 +2
- Misses 12149 12300 +151
- Partials 375 376 +1
Continue to review full report at Codecov.
|
8fca877
to
cc3e729
Compare
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
|
cc3e729
to
94dd63f
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, still needs code review
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, will be in metrics.yaml
- Is there a control mechanism that allows the user to turn the data collection on and off?
Yes, this telemetry is controlled by the Fenix 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, interaction with and success of primary password migration
- 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? (Yes/No) (If yes, set a todo reminder or file a bug if appropriate)**
No, expires 3/2021
- Does the data collection use a third-party collection tool? If yes, escalate to legal.
No
migration: | ||
type: event | ||
description: | | ||
Logins were successfully migrated using a master password. |
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.
Do we care about:
- number of people who clicked the banner and/or attempted to migrate
- failures to migrate (such as never entering the correct password, or some other failure?)
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.
cc @vesta0 for decision.
Users who clicked the banner is the "displayed" event.
Users who successfully migrated is the "migration" event.
Failures/dismissals are not currently separated.
bec90e3
to
36c24c1
Compare
607c8c9
to
1afde09
Compare
1afde09
to
2cfce3b
Compare
Pull Request checklist
To download an APK when reviewing a PR: