1097148 - locale detection to localized privacy policies isn't working

Status: RESOLVED FIXED

(www.mozilla.org :: Pages & Content, defect)

Description

[Jennifer Bertsch [:jbertsch]]

Hi-

It appears that the locale detection to redirect folks to a localized version of the privacy policy isn't working correctly. 

Tiles users on localized browsers are clicking on the privacy policy (see attached) and getting the en-us policy.  It looks like the right url is coded.

The privacy policies live in their special git repo.

Thx,
Jen

Comment 1

[Jennifer Bertsch [:jbertsch]]

Attachment: Screen Shot 2014-11-11 at 9.55.20 AM.png

Comment 2

[Jennifer Bertsch [:jbertsch]]

This is where the localized docs are on git https://github.com/mozilla/legal-docs/blob/master/firefox_privacy_notice/de.md

Comment 3

[Peiying Mo [:CocoMo]]

Can we fix all the docs, not just de?  AT least make sure all FF PN are linked correctly.  https://github.com/mozilla/legal-docs/tree/master/firefox_privacy_notice

Comment 4

[Paul [:pmac] McLanahan]

Committed to trunk in r134809.

Comment 5

[Paul [:pmac] McLanahan]

Committed to prod in r134811.

Comment 6

[Paul [:pmac] McLanahan]

As soon as caches clear, this should be fixed in prod.

Comment 7

[Jennifer Bertsch [:jbertsch]]

Pmac - It doesn't look fixed?  Thx.

Comment 8

[Paul [:pmac] McLanahan]

It looks fixed to me. I can see https://www.mozilla.org/de/privacy/firefox/ in German. If you can't it could be a browser cache thing on the /de/ -> /en-US/ redirect. Try in a private tab.

Comment 9

[Paul [:pmac] McLanahan]

Oh. I see in IRC that this was meant to include /privacy/ as well. That is not fixed but is the same issue. I'll push out those changes in a bit.

Comment 10

[Paul [:pmac] McLanahan]

Added lang files for main privacy page in r134830.

Comment 11

[Paul [:pmac] McLanahan]

Merged to prod in r134832.

This fixes most languages. The main one not fixed is "es" as in the legal-docs repo the "mozilla_privacy_policy" doc just has an "es.md" file, but no indication which "es" locale it should be on the site. bedrock doesn't support "es", but does support "es-ES", "es-AR", "es-MX" etc. So we should revisit this and probably file an issue with legal docs to get them to move this file into a proper local and provide other Spanish locales as well.

Comment 12

[Patrick Finch]

(In reply to Paul McLanahan [:pmac] from comment #11)
> Merged to prod in r134832.
> 
> This fixes most languages. The main one not fixed is "es" as in the
> legal-docs repo the "mozilla_privacy_policy" doc just has an "es.md" file,
> but no indication which "es" locale it should be on the site. bedrock
> doesn't support "es", but does support "es-ES", "es-AR", "es-MX" etc. So we
> should revisit this and probably file an issue with legal docs to get them
> to move this file into a proper local and provide other Spanish locales as
> well.

I think that 
https://www.mozilla.org/pt-PT/privacy/ should have the same content as
https://www.mozilla.org/pt-BR/privacy/
(At the moment it redirects to en-US)

The Spanish policy should be available for:
https://www.mozilla.org/es-AR/privacy/
https://www.mozilla.org/es-CL/privacy/
https://www.mozilla.org/es-ES/privacy/
https://www.mozilla.org/es-MX/privacy/

Comment 13

[Patrick Finch]

Is there an updated doc for Japanese?  Do we have any Japanese content on mozilla.org, or should that policy appear on:

http://www.mozilla.jp/legal/privacy/

Comment 14

[Francesco Lodolo [:flod]]

I think pt-BR and pt-PT are quite different in terms of language, anyhow we don't ship anything related to Firefox OS in pt-PT, so I believe we should just ignore it.

See Paul's comment above for es-*

See bug 1073206 for Japanese. As usual Japanese is in a very peculiar state, some pages are hosted on mozilla.jp, other on mozilla.org (see also bug 1040617).

Comment 15

[Rebecca Billings [:rbillings]]

Verified that the original issue is fixed- using a German browser, the privacy policy now displays correctly in German.

Comment 16

[Geoff Piper]

Adding Peiying to the bug as well (but I believe another Bug may need to be opened to cover all missing localizations/languages). Patrick please confirm either all languages are supported as needed, and if not work with Peiying whether in this bug or another as needed, to insure all languages intended are supported.

Once all languages are supported, localized/translated, they need to be supported in the browser the same as with these here so I expect the same process to apply to any and all additional languages. 

We need to insure the process of localization/translation then carries to proper tech implementation to make sure they display correctly in the correct locale/build etc.