1855096 - Add UTM tags to the web links in the core sidebar experience

Status: VERIFIED FIXED

(Firefox :: Shopping, enhancement, P1)

Description

[Ania]

In order to understand whether users click on the sidebar links to visit Fakespot's website in the sidebar footer, let's please add the following UTM tags to the Fakespot website in the core review checker experience:

https://www.fakespot.com/our-mission
utm_source=review-checker
utm_campaign=fakespot-by-mozilla
utm_medium=inproduct
utm_term=core-sidebar

and the SUMO page link:

https://support.mozilla.org/en-US/kb/review-checker-review-quality?as=u&utm_source=inproduct
utm_campaign=learn-more
utm_term=core-sidebar

Comment 1

[:Gijs (he/him)]

I'm a little confused about this ticket.

AIUI we already have client-side telemetry about these links, so do we definitely need the additional instrumentation?

Also, I believe SUMO handles its own utm tagging. The existing link has utm_source=inproduct if clicked from the sidebar. Could you confirm exactly what needs to change here?

Comment 2

[Ania]

Client-side telemetry does not overlap with Google Analytics data that all teams that work with SUMO to aggregate data on links visited in-product, so we do need the UTM tags.

Thank you for catching the discrepancy between how we were planning to catch utm_source/utm_medium for the SUMO article, that's my fault.
Let's then only update the Fakespot by Mozilla footer link (there was also a decision made to swap the link):

https://www.fakespot.com/review-checker
utm_source=review-checker
utm_campaign=fakespot-by-mozilla
utm_medium=inproduct
utm_term=core-sidebar

and the SUMO page link:

https://support.mozilla.org/en-US/kb/review-checker-review-quality?as=u&utm_source=inproduct
utm_campaign=learn-more
utm_term=core-sidebar

Comment 3

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

I think the right fix here is probably to use a plain link instead of the special sumo link, so that we can quickly fix this and get it uplifted. But we'll need to look at the moz-support-link reusable component to make sure we're attaching all the same utm params otherwise used by sumo.

Unassigning myself, as I'm out Monday and don't want to hold this up.

Comment 4

[Ania]

Accidentally updated the wrong ticket, my apologies

Comment 5

[Ania]

Attachment: Знімок екрана 2023-10-02 о 12.40.38 пп.png

Comment 6

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

input from hjones:

hmm yeah I suppose for a quick fix we could just add utm-campaign and utm-term as observedAttributes of moz-support-link

then just modify formatUTMParams slightly to accommodate those additional params: https://searchfox.org/mozilla-central/source/toolkit/content/widgets/moz-support-link/moz-support-link.mjs#113

that kind of extension should be totally backwards compatible. Longer term I think we’ll want to provide a cleaner way of passing multiple utm params to a link, rather than having to sprout/support an indefinite number of utm-* attrs…but that can be a follow up that we take on when it’s not time sensitive :-)

Comment 7

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

[Tracking Requested - why for this release]: We would like to add some additional UTM parameters to the SUMO links in the shopping sidebar.

Comment 8

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Attachment: WIP: Bug 1855096 - Add utm_campaign and utm_term support to moz-support-link WIP

sigh. not sure this will work, we do not have utm-content. just campaign and term. recheck tomorrow.

Comment 9

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Attachment: Bug 1855096 - Add UTM tags to the web links in the core sidebar experience r?#shopping-reviewers

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/31d0427deeca
Add UTM tags to the web links in the core sidebar experience r=shopping-reviewers,Gijs

Comment 11

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/31d0427deeca

Comment 12

[Rares Doghi, Desktop QA]

Hi @Jared, I'm a bit confused, what are the links that are being updated here ? are there multiple links ? is it the Learn more , privacy policy and terms links from the Onboarding card ?

Comment 13

[:Gijs (he/him)]

(In reply to Rares Doghi, Desktop QA from comment #12)

Hi @Jared, I'm a bit confused, what are the links that are being updated here ? are there multiple links ? is it the Learn more , privacy policy and terms links from the Onboarding card ?

It's the learn more link from the "How we determine review quality" section, and the "Fakespot by Mozilla" link at the bottom. This bug didn't change the links in the onboarding card.

Comment 14

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Comment on attachment 9357188 [details]
Bug 1855096 - Add UTM tags to the web links in the core sidebar experience r?#shopping-reviewers

Beta/Release Uplift Approval Request

• User impact if declined: No user impact, but product will not be able to distinguish visits to the shopping SUMO page and the fakespot.com info page that originate from the shopping sidebar.
• 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: As described in comment 13 of this bug (https://bugzilla.mozilla.org/show_bug.cgi?id=1855096#c13), click the "learn more" link from the "How we determine review quality" section, and the "Fakespot by Mozilla" link at the bottom below settings, and verify that the UTM parameters in the query string of the page reflect the parameters listed in comment 2 of the bug.
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This change changes two links in the shopping sidebar to set custom UTM parameters. This shopping sidebar will only be enabled for a small group of users enrolled via nimbus experiment, and the change introduced in this patch is extremely narrowly-scoped and covered by tests.
• String changes made/needed: None
• Is Android affected?: No

Comment 15

[Rares Doghi, Desktop QA]

This issue is verified as fixed in our latest Nightly build 120.0a1 (2023-10-10) however the Fakespot by mozilla link beneath the Settings now shows a 404 page is that ok ?

Comment 16

[:Gijs (he/him)]

(In reply to Rares Doghi, Desktop QA from comment #15)

This issue is verified as fixed in our latest Nightly build 120.0a1 (2023-10-10) however the Fakespot by mozilla link beneath the Settings now shows a 404 page is that ok ?

I think this is something to follow up with with the FS team. I'll raise it in slack.

Comment 17

[:Gijs (he/him)]

Comment on attachment 9357188 [details]
Bug 1855096 - Add UTM tags to the web links in the core sidebar experience r?#shopping-reviewers

Going to clear beta uplift request until we work out what's going on here per conversation in slack.

Comment 18

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

the fakespot URL should be https://www.fakespot.com/our-mission not https://www.fakespot.com/review-checker. Updating in a second patch before we request uplift.

Comment 19

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Attachment: Bug 1855096 - Correct the fakespot link URL shown at the bottom of the sidebar. r?#shopping-reviewers

Comment 20

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Reopening for the second patch, https://phabricator.services.mozilla.com/D190608

Comment 21

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/63c642970901
Correct the fakespot link URL shown at the bottom of the sidebar.

Comment 22

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Comment on attachment 9357188 [details]
Bug 1855096 - Add UTM tags to the web links in the core sidebar experience r?#shopping-reviewers

Beta/Release Uplift Approval Request

• User impact if declined: No user impact, but product will not be able to distinguish visits to the shopping SUMO page and the fakespot.com info page that originate from the shopping sidebar.
• 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: As described in comment 13 of this bug (https://bugzilla.mozilla.org/show_bug.cgi?id=1855096#c13), click the "learn more" link from the "How we determine review quality" section, and the "Fakespot by Mozilla" link at the bottom below settings, and verify that the UTM parameters in the query string of the page reflect the parameters listed in comment 2 of the bug. But note the fakespot.com link is to fakespot.com/our-mission.
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This change changes two links in the shopping sidebar to set custom UTM parameters. This shopping sidebar will only be enabled for a small group of users enrolled via nimbus experiment, and the change introduced in this patch is extremely narrowly-scoped and covered by tests.
• String changes made/needed: None
• Is Android affected?: Yes

Comment 23

[Rares Doghi, Desktop QA]

I was able to verify the fix in our latest Nightly build 120.0a1 (2023-10-10), the user is still redirected to the:
https://www.fakespot.com/review-checker?utm_source=review-checker&utm_campaign=fakespot-by-mozilla&utm_medium=inproduct&utm_term=core-sidebar
which still shows the 404 page, but if we replace the /review-checker with /our-mission, it will redirect to the correct page.

Comment 24

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/63c642970901

Comment 25

[Donal Meehan [:dmeehan]]

Comment on attachment 9357188 [details]
Bug 1855096 - Add UTM tags to the web links in the core sidebar experience r?#shopping-reviewers

Approved for 119.0b9

Comment 26

[Donal Meehan [:dmeehan]]

Comment on attachment 9357673 [details]
Bug 1855096 - Correct the fakespot link URL shown at the bottom of the sidebar. r?#shopping-reviewers

Approved for 119.0b9

Comment 27

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/745c87f92fd7
https://hg.mozilla.org/releases/mozilla-beta/rev/6f21dde739bd

Comment 28

[Rares Doghi, Desktop QA]

This issue is Verified as fixed in our latest NIghtly build as well as our latest Beta 119.0b9.