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

Status: RESOLVED FIXED

(support.mozilla.org :: Knowledge Base Content, task)

Description

[Jared Wein [:jaws] (please needinfo? me)]

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'

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

Mike, is this the right component for this and do you need any more information?

Comment 2

[Michael Cooper [:mythmon]]

This is a content issue. Joni can handle that. I'm not sure if she needs any more information.

Comment 3

[Joni Chan]

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.

Comment 4

[Gerald Squelart (he/him) (not at Mozilla since 2022-09-15)]

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.

Comment 5

[Joni Chan]

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.

Comment 6

[Jared Wein [:jaws] (please needinfo? me)]

Thanks Joni, I'll land changes to Firefox to use the VERSION/OS/LOCALE string in product.

Comment 7

[Jared Wein [:jaws] (please needinfo? me)]

Actually, we can just use this bug.

Comment 8

[Jared Wein [:jaws] (please needinfo? me)]

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.

Comment 9

[Gerald Squelart (he/him) (not at Mozilla since 2022-09-15)]

(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.