Links point to en-US URLs

VERIFIED FIXED in B2G C3 (12dec-1jan)

Status

P1
normal
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: willyaranda, Assigned: timdream)

Tracking

({l12y, late-l10n, regression})

unspecified
B2G C3 (12dec-1jan)
x86
Mac OS X
l12y, late-l10n, regression

Firefox Tracking Flags

(blocking-basecamp:+)

Details

Attachments

(1 attachment)

https://github.com/mozilla-b2g/gaia/blob/master/apps/browser/locales/browser.en-US.properties#L53

support-about.href=http://support.mozilla.org/1/firefox-os/1.0/firefox-os/en-US

faq-about.href=https://www.mozilla.org/en-US/firefox/mobile/faq/?os=firefox-os

They should not have any 'en-US' inside. Maybe the localeCode of the Gaia in that moment, or directly no language code, and then SUMO and Mozilla.org backends should redirect users, like it's done actually with other products.
Keywords: l12y, late-l10n

Comment 1

6 years ago
The got introduced in bug 802545, and it seems they didn't get proper l10n review at the time.

I request this to block basecamp. The FAQ link is just wrong, AFAICT.

Please change the urls to be locale agnostic, and remove them from the .properties files and move them to code. That's taking the most advantage out of the infrastructure we have server side, and allows us to adjust the URLs for the FAQ for example to something more focused once we have that.
blocking-basecamp: --- → ?
Depends on: 802545
Triage: BB+, C3, P1 - for l10n impact and regression
blocking-basecamp: ? → +
Keywords: regression
Priority: -- → P1
Target Milestone: --- → B2G C3 (12dec-1jan)
I can take it, but I need to know where should the strings be put? Vivien?
Assignee: nobody → timdream+bugs
Flags: needinfo?(21)
I would also need to know whether or not there should be two versions of URLs for official and unofficial build.
Pike, given the fact that Vivien has been unresponsive for 3 days, could you answer my questions above? Thanks.
Flags: needinfo?(l10n)
Like, what does Firefox do, as a prior-art?

Comment 7

6 years ago
Grace, is Firefox OS FAQ you?

The support ref should just pick the locale code from the runtime, and pass that along.
Flags: needinfo?(l10n) → needinfo?(gjimenez)
(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) from comment #3)
> I can take it, but I need to know where should the strings be put? Vivien?

Firefox usually use a preference for such things (http://mxr.mozilla.org/mozilla-central/source/browser/app/profile/firefox.js#856) The value of the preference contains some variables that are replaced at run time.

But reading comment 0 it seems like removing the en-US part of the url is enough.
Flags: needinfo?(21)
(In reply to Axel Hecht [:Pike] from comment #1)
> I request this to block basecamp. The FAQ link is just wrong, AFAICT.

Waiting for Grace answer here.
> 
> Please change the urls to be locale agnostic, and remove them from the
> .properties files and move them to code. That's taking the most advantage
> out of the infrastructure we have server side, and allows us to adjust the
> URLs for the FAQ for example to something more focused once we have that.

How does moving them out of the properties change anything on the infrastructure on the server side?
Created attachment 695938 [details]
Github: https://github.com/mozilla-b2g/gaia/pull/7202

Let's not worry about the official/unofficial thing in this bug then.
Attachment #695938 - Flags: review?(community)
Attachment #695938 - Flags: review?(alive)
Attachment #695938 - Flags: review?(community) → review?(l10n)
(In reply to Vivien Nicolas (:vingtetun) from comment #9)
> (In reply to Axel Hecht [:Pike] from comment #1)
> > I request this to block basecamp. The FAQ link is just wrong, AFAICT.
> 
> Waiting for Grace answer here.

Is http://support.mozilla.org/products/firefox-os the right address? If so you could take my patch as-is.
Comment on attachment 695938 [details]
Github: https://github.com/mozilla-b2g/gaia/pull/7202

r=me. Powerful l10n lib I never know it has the ability to replace DOM attribute \O/ https://github.com/mozilla-b2g/gaia/blob/master/shared/js/l10n.js#L242
Attachment #695938 - Flags: review?(alive) → review+
Comment on attachment 695938 [details]
Github: https://github.com/mozilla-b2g/gaia/pull/7202

Sounds good enough for me. Grace if you're unhappy please open a followup bug.
Attachment #695938 - Flags: review?(l10n) → review+
In Unagi build 20121231070201, the URL for the User Guide page is http://support.mozilla.org/en-US/products/firefox-os?as=u&utm_source=inproduct.  This is the case even after the language and location are set to a non-US location.  Reopening.
Status: RESOLVED → REOPENED
Flags: needinfo?(gjimenez)
Resolution: FIXED → ---
(In reply to ckreinbring from comment #15)
> In Unagi build 20121231070201, the URL for the User Guide page is
> http://support.mozilla.org/en-US/products/firefox-
> os?as=u&utm_source=inproduct.  This is the case even after the language and
> location are set to a non-US location.  Reopening.

Please file another bug and clearly states where the link is.

This bug is about the "Support" and "FAQ" link in the Browser app.
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED
(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) from comment #16)
> (In reply to ckreinbring from comment #15)
> > In Unagi build 20121231070201, the URL for the User Guide page is
> > http://support.mozilla.org/en-US/products/firefox-
> > os?as=u&utm_source=inproduct.  This is the case even after the language and
> > location are set to a non-US location.  Reopening.
> 
> Please file another bug and clearly states where the link is.
> 
> This bug is about the "Support" and "FAQ" link in the Browser app.

Bug 825831 filed.

Comment 18

6 years ago
Verified it's fixed on "Unagi"
Build ID:20130114073222

Mozilla support page changes language according to the language chosen in "Language" in the settings.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.