Pocket only enabled on en-US, hard-coded locales aren't picked up

RESOLVED FIXED in Firefox 38.0.5

Status

()

defect
--
critical
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Pike, Assigned: Dolske)

Tracking

38 Branch
Firefox 41
All
Unspecified
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox38.0.5+ verified, firefox39 fixed, firefox40 fixed, firefox41 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

4 years ago
the beta repository still has

pref("browser.pocket.enabledLocales", "en-US");

so the localizations we landed aren't shown in the UI at this point.

flod and Jeff verified with the 38.0.5 beta candidates from today.
Reporter

Comment 1

4 years ago
[Tracking Requested - why for this release]:
Assignee

Comment 2

4 years ago
Sigh. Yes, this got missed. I think we were assuming we'd have a Nightly to verify all the L10N stuff before enabling, but then we YOLO-landed everything to make the schedule. :(

Patch is trivial, will take "what do we do for beta1" to a release-drivers thread.
Assignee

Comment 3

4 years ago
Posted patch Patch v.1Splinter Review
Assignee: nobody → dolske
Attachment #8604168 - Flags: review?(adw)

Updated

4 years ago
Attachment #8604168 - Flags: review?(adw) → review+
Assignee

Comment 4

4 years ago
Landed on m-r for an immediate respin for 38.0.5 beta1, build2. 

https://hg.mozilla.org/releases/mozilla-release/rev/fff143cacb66
QA Contact: andrei.vaida
Tested Spanish again, I have a Pocket button and strings localized in Spanish in the UI.
https://hg.mozilla.org/mozilla-central/rev/569d3999242d
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Verified that de, es-Es, ja and ru locales have Pocket enabled using Firefox 38.0.5 beta 1 build 2 under Win 7 64-bit, Ubuntu 14.04 32-bit and Mac OS X 10.9.5.

Please note that ja locale doesn't have Pocket installed under Mac OSX, probably because the build here is named ja-JP-mac.
Also under OS X, the elements in the Pocket drop-down for the ru build are not aligned.

There is no icon for other locales.
Justin, can you please take a look at the above comment and let us know the next actions?
Flags: needinfo?(dolske)
Reporter

Comment 10

4 years ago
Petruta, are bugs filed for those yet?

Also, for ja-JP-mac, it'd be interesting to know if fixing browser.pocket.enabledLocales to include ja-JP-mac fixes the issue. I wonder if the pocket strings would need adjustments, too.
Assignee

Updated

4 years ago
Flags: needinfo?(dolske)
Assignee

Comment 11

4 years ago
Comment on attachment 8604168 [details] [diff] [review]
Patch v.1

[Triage Comment]

a+ for aurora/beta/release: required for Pocket launch in 38.0.5.
Attachment #8604168 - Flags: approval-mozilla-release+
Attachment #8604168 - Flags: approval-mozilla-beta+
Attachment #8604168 - Flags: approval-mozilla-aurora+
Assignee

Comment 12

4 years ago
https://hg.mozilla.org/releases/mozilla-beta/rev/dc92613a9ba6

(previously landed on m-r -- comment 4, not yet landed on m-a due to tree closure)
You need to log in before you can comment on or make changes to this bug.