Closed Bug 1592107 Opened 6 years ago Closed 6 years ago

Update utm term utm_campaign value by appending channel name for first run about:welcome experiments

Categories

(Firefox :: Messaging System, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
Firefox 72
Iteration:
72.1 - Oct 21 - Nov 3
Tracking Status
firefox71 + verified
firefox72 --- verified

People

(Reporter: pdahiya, Assigned: pdahiya)

References

Details

Attachments

(1 file, 1 obsolete file)

about:welcome experiments in 71 are controlled from firefox not normandy, and the normal build process makes the very late beta/rcs appear like a regular release build.

To indicate to amplitude requests coming from first run experiments are from specific channel beta vs release — append channel in a utm term utm_campaign e.g. utm_campaign=firstrun-beta

As of now utm_campaign value is set to 'firstrun' and this change will update utm_campaign value to 'firstrun-<channel>' where channel is value such as nightly, beta or release picked from pref 'app.update.channel'

Summary: Update utm term utm_campaign value by appending channel name → Update utm term utm_campaign value by appending channel name for first run about:welcome experiments

Is there a good way to get / pass the release channel to asrouter content? If it doesn't already exist, I suppose app.update.channel could be exposed as a redux Pref value ?

Flags: needinfo?(andrei.br92)

That sounds like the easiest way to go.

Flags: needinfo?(andrei.br92)
Assignee: nobody → pdahiya
Iteration: --- → 72.1 - Oct 21 - Nov 3
Priority: -- → P1
Pushed by pdahiya@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f3e7b3551c54 Append channel name to utm_campaign parameter for about:welcome experiments r=Mardak

Steps for QA:
1. Create a new profile
2. Verify trailhead.firstrun.branches is empty
3. Set app.normandy.user_id to one of the values https://bugzilla.mozilla.org/show_bug.cgi?id=1590504#c3 to trigger respective about:welcome variant.
4. Restart and open about:welcome
5. Click on links on about:welcome page and verify utm_campaign parameter value sent in url opened is set to 'firstrun-<channel>' where channel is value from pref 'app.update.channel'
6. Enter email address and trigger Signin or Signup flow and verify utm_campaign value in the url is set correctly. Validate user is able to successfully complete signin and signup
7. Verify click on triplets cards on new tab and full page sends utm_campaign value as 'firstrun-<channel>'

Trigger above steps and verify for both modal and full page variant of about:welcome

Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72

[Tracking Requested - why for this release]:
To support 71 first run about:welcome experiments and differentiate users by channel in FxA data analysis in Amplitude.

I have verified this issue with the steps provided in Comment 5 on Latest Firefox Nightly 72.0a1 (Build ID: 20191103213857) on Windows 10 x64, Mac 10.14.6 and Arch Linux.
The utm_campaign value is correctly set to the value from the app.update.channel preference when triggering the Signup flow, and when clicking on the triplet cards both from the modals and the full page variants.

Status: RESOLVED → VERIFIED

Is attachment 9106382 [details] the same as the one that landed in m-c? If so, there doesn't need to be a separate patch for mozilla-beta if it applies cleanly to mozilla-beta already.

Flags: needinfo?(pdahiya)

(In reply to Ed Lee :Mardak from comment #11)

Is attachment 9106382 [details] the same as the one that landed in m-c? If so, there doesn't need to be a separate patch for mozilla-beta if it applies cleanly to mozilla-beta already.

Wasn't sure as patch includes bundle file, just verified https://phabricator.services.mozilla.com/D50968 applies cleanly to beta. Will abandon 9106382 and submit beta approval request for D50968.

Flags: needinfo?(pdahiya)
Attachment #9106382 - Attachment is obsolete: true

Comment on attachment 9104963 [details]
Bug 1592107 - Append channel name to utm_campaign parameter for about:welcome experiments

Beta/Release Uplift Approval Request

  • User impact if declined: Impacts data analysis in FxA, patch helps to differentiate late beta users from release users
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: https://bugzilla.mozilla.org/show_bug.cgi?id=1592107#c5
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Specific to data analysis and changes existing utm parameter value by appending channel name from pref 'app.update.channel'
  • String changes made/needed: None
Attachment #9104963 - Flags: approval-mozilla-beta?

Comment on attachment 9104963 [details]
Bug 1592107 - Append channel name to utm_campaign parameter for about:welcome experiments

Low risk, covered by tests, uplift approved for 71 beta 8, thanks.

Attachment #9104963 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
QA Whiteboard: [qa-triaged]

I have verified this issue with the steps provided in Comment 5 on Firefox Beta 71.0b8 (Build ID: 20191107101713) on Windows 10 x64, Mac 10.14.6 and Arch Linux.
The utm_campaign value is correctly set to the value from the app.update.channel preference when triggering the Signup flow, and when clicking on the triplet cards both from the modals and the full page variants.

Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: