Closed Bug 1706489 Opened 4 years ago Closed 4 years ago

Allow for calculating upgrade dialog funnel by first screen primary action

Categories

(Firefox :: Messaging System, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
90 Branch
Iteration:
90.1 - Apr 19 - May 2
Tracking Status
firefox89 --- verified
firefox90 --- verified

People

(Reporter: Mardak, Assigned: Mardak)

References

Details

(Whiteboard: [proton-onboarding][proton-uplift])

Attachments

(1 file)

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.

Reuse existing "show" event object to track which primary button was shown on the first screen.

Pushed by elee@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3ac594597045 Allow for calculating upgrade dialog funnel by first screen primary action r=nanj

Backed out for causing browser-chrome failures in browser_browserGlue_upgradeDialog

Backout link: https://hg.mozilla.org/integration/autoland/rev/7f33f31f0e384d3696cae657be3f40e5ebb03600

Push with failures

Failure log

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

Flags: needinfo?(edilee)
Pushed by elee@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/73ef5130bfff Allow for calculating upgrade dialog funnel by first screen primary action r=nanj
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch

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
  1. ensure browser.startup.upgradeDialog.version is empty
  2. restart
  3. see upgrade dialog and click the blue primary button
  4. 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
Flags: needinfo?(edilee)
Attachment #9217227 - Flags: approval-mozilla-beta?
Flags: qe-verify+
QA Whiteboard: [qa-triaged]

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.

Status: RESOLVED → VERIFIED
Whiteboard: [proton-uplift]

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.

Attachment #9217227 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

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.

Flags: qe-verify+
Whiteboard: [proton-uplift] → [proton-onboarding][proton-uplift]
Blocks: 1710460
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: