-
Notifications
You must be signed in to change notification settings - Fork 1.3k
#10824: Onboarding telemetry #11867
#10824: Onboarding telemetry #11867
Conversation
app/metrics.yaml
Outdated
@@ -122,7 +122,53 @@ events: | |||
notification_emails: | |||
- fenix-core@mozilla.com | |||
expires: "2020-09-01" | |||
preference_toggled: | |||
whats_new_tapped: |
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.
This ordering got all messed up. I pulled preference_toggled
down into it's own section (per discussion with @sblatz and Marissa) and nested all onboarding clicks within that
Codecov Report
@@ Coverage Diff @@
## master #11867 +/- ##
============================================
- Coverage 22.11% 22.04% -0.08%
+ Complexity 727 726 -1
============================================
Files 377 377
Lines 15121 15153 +32
Branches 1961 1965 +4
============================================
- Hits 3344 3340 -4
- Misses 11491 11527 +36
Partials 286 286
Continue to review full report at Codecov.
|
@eliserichards can you include a data review form please? |
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.
|
d612c71
to
9c5ff3e
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.
Looks good overall to me, just a couple comments!
6793da2
to
6a82259
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.
These all look great from an eng standpoint :)
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, interaction data with onboarding is in metrics.md
- Is there a control mechanism that allows the user to turn the data collection on and off?
Yes, included under Fenix data settings
- 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, user interaction with onboarding cards
- 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, interaction data only
- 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?
No, has expiry
- Does the data collection use a third-party collection tool?
no
pref_toggled_private_browsing: | ||
type: event | ||
description: | ||
The private browsing preference was selected from the onboarding card. |
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.
Does this automatically trigger the PB mode being saved to the metrics ping for private browsing setting? https://github.com/mozilla-mobile/fenix/blob/master/docs/metrics.md#metrics-1
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.
Yes it does!
For #10824
Events:
This PR is covering:
Already exists in preference metrics
Pull Request checklist
After merge
To download an APK when reviewing a PR: