[IPProtection] Update the sumo link
Categories
(Firefox for Android :: Privacy, task)
Tracking
()
| 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
| Assignee | ||
Updated•14 days ago
|
Updated•14 days ago
|
| Assignee | ||
Comment 1•13 days ago
|
||
Updated•13 days ago
|
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?
| Assignee | ||
Comment 5•7 days ago
|
||
| Assignee | ||
Comment 6•7 days ago
|
||
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!
| Assignee | ||
Comment 8•6 days ago
|
||
Fixing the product in the link, support/firefox to support/mobile
Comment 9•6 days ago
|
||
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)
Comment 10•2 days ago
|
||
Comment 11•2 days ago
|
||
| bugherder | ||
Description
•