Allow for calculating upgrade dialog funnel by first screen primary action
Categories
(Firefox :: Messaging System, enhancement, P1)
Tracking
()
People
(Reporter: Mardak, Assigned: Mardak)
References
Details
(Whiteboard: [proton-onboarding][proton-uplift])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details |
The first screen has 4 possible primary actions:
- primary
- default
- pin
- theme
We already have event telemetry from bug 1697222 when users click on a given button but don't know how many times a button is dynamically shown.
Assignee | ||
Comment 1•4 years ago
|
||
Reuse existing "show" event object to track which primary button was shown on the first screen.
Comment 3•4 years ago
|
||
Backed out for causing browser-chrome failures in browser_browserGlue_upgradeDialog
Backout link: https://hg.mozilla.org/integration/autoland/rev/7f33f31f0e384d3696cae657be3f40e5ebb03600
INFO - TEST-PASS | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | parent must be in snapshot. Has [parent]. - true == true -
[task 2021-04-21T05:36:03.280Z] 05:36:03 INFO - Buffered messages finished
[task 2021-04-21T05:36:03.280Z] 05:36:03 INFO - TEST-UNEXPECTED-FAIL | browser/components/tests/browser/browser_browserGlue_upgradeDialog.js | After filtering we must have the expected number of events. - 4 == 3 - JS frame :: resource://testing-common/TelemetryTestUtils.jsm :: assertEvents :: line 177
Comment 5•4 years ago
|
||
bugherder |
Assignee | ||
Comment 6•4 years ago
|
||
Comment on attachment 9217227 [details]
Bug 1706489 - Allow for calculating upgrade dialog funnel by first screen primary action r?pdahiya,nanj
Beta/Release Uplift Approval Request
- User impact if declined: Inability to determine click-through rates of upgrade dialog primary button
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: 1. set browser.startup.homepage_override.mstone to 88
- ensure browser.startup.upgradeDialog.version is empty
- restart
- see upgrade dialog and click the blue primary button
- check about:telemetry#events-tab for upgrade_dialog#content#show#<button-displayed>
- List of other uplifts needed: none
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Adding one event telemetry without changing user visible behavior or designs
- String changes made/needed: none
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Comment 7•4 years ago
|
||
I have verified this enhancement using the steps from comment 6 and I can confirm that the "upgrade_dialog#content#show#upgrade-dialog-new-primary-default-button" event is successfully displayed in the "about:telemetry#events-tab" page.
Verified using the latest Firefox Nightly (90.0a1 Build ID - 20210422213157) installed on Windows 10 x64, Mac 10.15.7, and Linux Mint 20 x64.
Updated•4 years ago
|
Comment 8•4 years ago
|
||
Comment on attachment 9217227 [details]
Bug 1706489 - Allow for calculating upgrade dialog funnel by first screen primary action r?pdahiya,nanj
Approved for 89 beta 5, thanks.
Updated•4 years ago
|
Comment 9•4 years ago
|
||
bugherder uplift |
Comment 10•4 years ago
|
||
I have verified this enhancement using the steps from comment 6 and I can confirm that the "upgrade_dialog#content#show#upgrade-dialog-new-primary-default-button" event is successfully displayed in the "about:telemetry#events-tab" page.
Verified using the latest Firefox Beta (89.0b5 Build ID - 20210427185821) installed on Windows 10 x64, Mac 10.15.7, and Linux Mint 20 x64.
Assignee | ||
Updated•4 years ago
|
Description
•