Closed Bug 1038650 Opened 5 years ago Closed 5 years ago
[cy, eu, ff, gd, hy-AM, is] Add Fx32 locales to mobile/android/locales/maemo-locales
Please add cy, eu, ff, gd, hy-AM and is to mozilla-aurora/mobile/android/locales/maemo-locales. All locales have signed off and are ready to go. Please port to Beta at merge day as well.
Approval Request Comment [Feature/regressing bug #]:1019723, 639847, 1019740, 1019708, 1019800, 1035990 [User impact if declined]: Inavailability of these six new locales to users who speak those languages. [Describe test coverage new/current, TBPL]: L10n teams have undergone thorough testing of single-locale builds before signing off on their own devices. They enlisted the help of their local communities and users to receive feedback and have updated the translation quality along the way. rnewman created a multilocale build containing most of these to test them in that setting as well. [Risks and why]: Testing has revealed that many, if not all, popular Android devices are able to support these locales. Our testing has been extensive to attempt to discover any risk and address it. If any remains, it is very low. [String/UUID change made/needed]: N/A
Comment on attachment 8456107 [details] [diff] [review] add-fx32-locales.patch Review of attachment 8456107 [details] [diff] [review]: ----------------------------------------------------------------- lgtm. mfinkle can intervene later if he disagrees!
Attachment #8456107 - Flags: review?(mark.finkle) → review+
Comment on attachment 8456107 [details] [diff] [review] add-fx32-locales.patch Aurora+
Attachment #8456107 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
I've created the tests in Moztrap: cy: https://moztrap.mozilla.org/manage/case/14019 eu: https://moztrap.mozilla.org/manage/case/14020 ff: https://moztrap.mozilla.org/manage/case/14021 gd: https://moztrap.mozilla.org/manage/case/14022 hy-AM: https://moztrap.mozilla.org/manage/case/14023 is: https://moztrap.mozilla.org/manage/case/14024
Flags: in-moztrap?(fennec) → in-moztrap+
You need to log in before you can comment on or make changes to this bug.