Need SUMO redirect for 'fix-video-audio-problems-firefox-windows'

RESOLVED FIXED

Status

support.mozilla.org
Knowledge Base Content
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jaws, Assigned: jaws)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

For bug 1160424, we need a SUMO redirect for 'fix-video-audio-problems-firefox-windows' to redirect to 'https://support.mozilla.org/kb/fix-video-audio-problems-firefox-windows'
Component: Audio/Video: Playback → Knowledge Base Software
Product: Core → support.mozilla.org
Mike, is this the right component for this and do you need any more information?
Flags: needinfo?(mcooper)
This is a content issue. Joni can handle that. I'm not sure if she needs any more information.
Component: Knowledge Base Software → Knowledge Base Content
Flags: needinfo?(mcooper) → needinfo?(jsavage)
Hi Jared, I've set up a redirect to https://support.mozilla.org/kb/fix-video-audio-problems-firefox-windows. 

I don't know what the code means in bug 1160424, but the URL that's being redirected is  https://support.mozilla.org/1/firefox/%VERSION%/%OS%/%LOCALE%/fix-video-audio-problems-firefox-windows. 

Please "need info" me if there's anything else I can do.
Flags: needinfo?(jsavage)
Joni,

As per bug 1160424 comment 21 we expected to just go to
  https://support.mozilla.org/fix-video-audio-problems-firefox-windows
(notice there is no 'kb/'), and be automatically redirected to
  https://support.mozilla.org/kb/fix-video-audio-problems-firefox-windows
I.e., you'd add the missing 'kb/'.

The non-kb URL currently shows "Page Not Found" and an angry Lego dude.

Now, if our assumption was incorrect and you believe we should go directly to the 'kb/' page instead, please let us know.
Flags: needinfo?(jsavage)
Hi Gerald,

For in-product links, I suggest that we follow this format: 

https://support.mozilla.org/1/firefox/%VERSION%/%OS%/%LOCALE%/fix-video-audio-problems-firefox-windows

Alternatively, you can just add the missing 'kb/' to your original link, although if anyone changes the URL on our end, you'd have to change the hard-coded URL on your end to match the new URL.

The in-product link format would let SUMO change redirects without any code changes on your end.
Flags: needinfo?(jsavage)
Thanks Joni, I'll land changes to Firefox to use the VERSION/OS/LOCALE string in product.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Actually, we can just use this bug.
Assignee: nobody → jaws
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Okay, I looked in to this more and we should be creating the correct URL (with version, os, and locale). We use Services.urlFormatter.formatURLPref("app.support.baseURL") and that includes version, os, and locale, which get replaced by formatURLPref.

Gerald, maybe this was just a miscommunication, but have you seen the URL actually get created by Firefox that doesn't redirect properly? Note that the mochitest may be confusing because the pref used by tests doesn't include all of the variables.
Flags: needinfo?(gsquelart)
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #8)
> Okay, I looked in to this more and we should be creating the correct URL
> (with version, os, and locale). We use
> Services.urlFormatter.formatURLPref("app.support.baseURL") and that includes
> version, os, and locale, which get replaced by formatURLPref.
> 
> Gerald, maybe this was just a miscommunication, but have you seen the URL
> actually get created by Firefox that doesn't redirect properly? Note that
> the mochitest may be confusing because the pref used by tests doesn't
> include all of the variables.

I saw it while running a local build.

I've just tried on Nightly 49.0a1 2016-04-28, on Windows N... And it works!
It first goes to https://support.mozilla.org/1/firefox/49.0a1/WINNT/en-US/fix-video-audio-problems-firefox-windows
which gets redirected to https://support.mozilla.org/en-US/kb/fix-video-audio-problems-firefox-windows?as=u&utm_source=inproduct

So it's all good!

This is annoying that local builds don't go to the right place. :-(
Sorry for the panic.
Status: REOPENED → RESOLVED
Last Resolved: 2 years ago2 years ago
Flags: needinfo?(gsquelart)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.