Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[Bug] [L10N] Privacy notice is not translated in Korean #4216

Closed
lobontiumira opened this issue Jul 22, 2019 · 7 comments
Closed

[Bug] [L10N] Privacy notice is not translated in Korean #4216

lobontiumira opened this issue Jul 22, 2019 · 7 comments
Labels
🐞 bug Crashes, Something isn't working, .. 🌐 L10N Localization, translation, strings, ..

Comments

@lobontiumira
Copy link

lobontiumira commented Jul 22, 2019

Steps to reproduce

  1. Set your device's language to Korean.
  2. Go to Settings - Privacy notice and observe the text.
  3. Tap on it and observe the Privacy notice page.

Expected behavior

Both option and page should be translated.

Actual behavior

The option from Settings and the Privacy notice page are not translated.

Note

In Japanese, only the option Privacy notice from Settings is not translated, the actual page is.

Device information

  • Android device: Xiaomi MiPad2 (Android 5.1) - x86 architecture
  • Fenix version: RC v1.1.0-rc.1

korean
korean1

┆Issue is synchronized with this Jira Task

@lobontiumira lobontiumira added 🐞 bug Crashes, Something isn't working, .. 🌐 L10N Localization, translation, strings, .. labels Jul 22, 2019
@sblatz
Copy link
Contributor

sblatz commented Jul 22, 2019

Don't we depend on contributors for these translations? It's probably just not been picked up yet.

@Delphine can you provide confirmation?

@Delphine
Copy link
Contributor

Delphine commented Jul 23, 2019

Thanks for flagging. Handing over to my team mate @peiying2 since I'm in charge of mobile l10n, and the Privacy notice page actually lives here: https://github.com/mozilla/legal-docs/tree/master/firefox_privacy_notice

@peiying2
Copy link

@sblatz it is true that this document is not localized in Korean. In fact, the document is not localized in most of the languages that Firefox desktop and mobile support. The current policy is to support top 12 (desktop) locales by default. For this doc, the rest of the locales are on an older version and are redirecting users to the English version.
For Mobile products, so far we localize a few locales based on markets (e.g. Firefox Lite). If Korean must be added to the Fx PN, per requirement, I will discuss this with Legal counsel. Please advise.

@sblatz
Copy link
Contributor

sblatz commented Jul 23, 2019

Thanks @peiying2 for the info :) will discuss with product!

@peiying2
Copy link

Thanks @peiying2 for the info :) will discuss with product!

The decision by legal to support top 12 is purely a budget issue. If Korean must be added, is this the only language?

@kbrosnan
Copy link
Contributor

This appears to be expected behavior. Should we close this?

@sblatz
Copy link
Contributor

sblatz commented Jul 26, 2019

Agreed. Please open a ticket if we want Korean to be a "must"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. 🌐 L10N Localization, translation, strings, ..
Projects
None yet
Development

No branches or pull requests

6 participants