Closed
Bug 1251749
Opened 9 years ago
Closed 9 years ago
Use UI telemetry to tell if settings are enabled or not
Categories
(Firefox for Android Graveyard :: Metrics, defect)
Firefox for Android Graveyard
Metrics
Tracking
(firefox48 fixed)
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: Margaret, Assigned: Margaret)
References
Details
Attachments
(1 file)
Right now we use UI telemetry to tell whether or not people tap on items in settings, but that's not actually what we want to know.
We use histogram probes to collect that value of certain individual settings, but it's burdensome to add that for every new setting, as well as expensive on data to continue to add histograms.
We should change our current settings UI telemetry to indicate the value of the setting when it's changed. Mark, do you know what we could send to make analysis simple here?
Flags: needinfo?(mark.finkle)
Comment 1•9 years ago
|
||
I accidentally commented about this very thing here: https://bugzilla.mozilla.org/show_bug.cgi?id=1251454#c2
Flags: needinfo?(mark.finkle)
Assignee | ||
Comment 2•9 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #1)
> I accidentally commented about this very thing here:
> https://bugzilla.mozilla.org/show_bug.cgi?id=1251454#c2
For completeness:
(In reply to Mark Finkle (:mfinkle) from comment #2)
> I would like us to move away from histograms and look for event data ways to
> capture this data. I think it's more useful to look at what people are
> changing in relation to other events. Examples:
>
> 1. Saw onboarding talk about data saver, and soon after turn it on (never or
> wifi)
> 2. Saw a prompt for Tab Queues and turned it on (boolean)
> 3. Saw a notification for updated content, didn't like it and turned it off
> (via the settings button)
>
> We track event probes for any changes to preferences: (edit.1, settings,
> name-of-pref). I wonder how useful it would be to change to: (edit.1,
> settings, ["name-of-pref", "value"])
>
> where value would be (0, 1) for booleans, integer for modes, and "*" for
> user entered strings? I used "*" because we need to be careful with data
> that could uniquely identify someone.
>
> SQL and Spark are both capable of "unpacking" the array, if needed.
Let's write a patch! This is a single line that needs changing:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/preferences/GeckoPreferences.java#1206
Assignee: nobody → margaret.leibovic
Assignee | ||
Updated•9 years ago
|
Assignee: margaret.leibovic → gkruglov
Assignee | ||
Comment 4•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/43589/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/43589/
Attachment #8736911 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 5•9 years ago
|
||
(In reply to :Margaret Leibovic from comment #2)
> > where value would be (0, 1) for booleans, integer for modes, and "*" for
> > user entered strings? I used "*" because we need to be careful with data
> > that could uniquely identify someone.
Many of our settings are actually string types, so I didn't want to store "*" for all strings. I did an audit of the settings to look for places where users can enter text, and I believe the homepage preference is the only place, so I special-cased that.
Assignee: gkruglov → margaret.leibovic
Comment 6•9 years ago
|
||
Comment on attachment 8736911 [details]
MozReview Request: Bug 1251749 - Use UI telemetry to tell if settings are enabled or not. r=sebastian
https://reviewboard.mozilla.org/r/43589/#review40619
Attachment #8736911 -
Flags: review?(s.kaspari) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 7•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/71d3427c683190cdb409063ad67e24d5926f4591
Bug 1251749 - Use UI telemetry to tell if settings are enabled or not. r=sebastian
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 8•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Updated•5 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•