Closed Bug 2042931 Opened 14 days ago Closed 13 days ago

[IPProtection] Update the sumo link

Categories

(Firefox for Android :: Privacy, task)

All
Android
task

Tracking

()

RESOLVED FIXED
153 Branch
Tracking Status
firefox153 --- fixed

People

(Reporter: mavduevskiy, Assigned: mavduevskiy, NeedInfo)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxdroid] [group2])

Attachments

(3 files)

The sumo link was updated from firefox-android-vpn to mobile-android-vpn (firefox is typically used for desktop, as QA insightfully pointed out).

We should update the corresponding SumoLink

Whiteboard: [fxdroid] [group2]
Assignee: nobody → mavduevskiy
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 13 days ago
Resolution: --- → FIXED
Target Milestone: --- → 153 Branch

Hi Mike, I'm still seeing /firefox in the SUMO URL (https://support.mozilla.org/1/firefox/153.0a1/Android/en-US/mobile-android-vpn). Just wanted to double-check, wasn't that one supposed to change too?

Flags: needinfo?(mavduevskiy)
Flags: needinfo?(mavduevskiy)

hi! when I open a link, it navigates correctly to mobile android vpn. could you record a video?
I removed the firefox-android-vpn from the code base (existing link), so it should be fine?

Ah, looking back at the Slack thread, I see where the mix-up happened!
The change to mobile-android-vpn at the end of the URL is perfect. However, the product tag in the middle accidentally remained to /firefox/ instead of /mobile/:
https://support.mozilla.org/1/ firefox /1.0.2623/Android/en-GB/mobile-android-vpn
Per Flavius's note in the chat, the middle /firefox/ tag tracks desktop products, so we need that middle slot to be as /mobile/ so it tracks mobile metrics correctly.
Could we update the middle segment to /mobile/? No video needed since the page destination itself works great!

Flags: needinfo?(mavduevskiy)

Fixing the product in the link, support/firefox to support/mobile

A patch has been attached on this bug, which was already closed. Filing a separate bug will ensure better tracking. If this was not by mistake and further action is needed, please alert the appropriate party. (Or: if the patch doesn't change behavior -- e.g. landing a test case, or fixing a typo -- then feel free to disregard this message)

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: