Closed
Bug 925787
Opened 11 years ago
Closed 11 years ago
separate fallback locale for fhr mobile from fhr desktop
Categories
(Firefox Health Report Graveyard :: Web: Health Report, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: pascalc, Assigned: pascalc)
References
Details
(Whiteboard: [android][qa+])
Attachments
(1 file)
We have fhr ready for almost all locales on Desktop but not for mobile, for example fhr Desktop translation in Polish is done while fhr mobile isn't yet. We have a lot more people involved on Desktop l10n than on mobile l10n actually so that's unlikely to change in the short term.
Currently it's a all or nothing situation, if we want to offer a fallback to English, that will make Desktop fallback to English too even though it's localized and shipped.
I am going to propose a patch for generate.py to fix that and allow us to have a locale activated for Desktop but with a separate fallback to another locale for mobile fhr.
Assignee | ||
Comment 1•11 years ago
|
||
Pointer to Github pull-request
Comment 2•11 years ago
|
||
Marking resolved since this pr was merged ages ago.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Whiteboard: [android][qa+]
Comment 3•11 years ago
|
||
Suggestions for how to test this; I'm not entirely sure how to identify where (which locale) the fallback will occur?
Comment 4•11 years ago
|
||
Wow, this bug is old, and the mobile version never made it to production :-\
Then only fallback is Japanese right now.
https://github.com/pascalchevrel/fhr-jelly/blob/master/settings.py#L64
So, if you visit dev with a Japanese phone, you should see en-US content.
Comment 6•11 years ago
|
||
Oops .. didn't mean to ?needsinfo? you pascalc. Flod has me covered.
Bumping to QA verified on dev for /ja/ on Android. The fallback to /en-US/ correctly occurs. Looks like were ready for a prod release :)
Status: RESOLVED → VERIFIED
Flags: needinfo?(pascalc)
Updated•6 years ago
|
Product: Firefox Health Report → Firefox Health Report Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•