Closed Bug 1164161 Opened 8 years ago Closed 8 years ago

Panel dictionary file missing entries for some languages


(Firefox :: Pocket, defect)

Not set



Firefox 41
Tracking Status
firefox38.0.5 --- fixed
firefox39 --- fixed
firefox40 --- fixed
firefox41 --- fixed


(Reporter: nate+bugzilla, Assigned: nate+bugzilla)



(1 file)

Peter noted that we had some conflicts in dictionary.js:

We found that non-English languages were missing or had keys not matching up for 3 strings:
- saving: "Saving..." (Missing)
- mustbeconnected: "You must be connected to the Internet in order to save to Pocket. Please check your connection and try again." (Missing) 
- errorgeneric: "There was an error when trying to save to Pocket." (Not matching)

Note, this isn't a last minute Beta blocker as the fallback will be not to show any string. In the three cases above it's okay because:
- saving: It shows a spinner, actually looks normal without it
- mustbeconnected: Still has the main header of the error dialog that says "Page Not Saved" -- it just won't explain why.
- errorgeneric: Very rare error state, only will happen if Pocket is completely knocked out. Still has the main header of the error dialog that says "Page Not Saved" -- it just won't explain why.

We had strings for those from our own translations so I dropped them in. That said, I assume you'll want to have your localization team review/update as needed. But hopefully this let's us move quickly if needed.
Attachment #8604828 - Flags: review?(jaws)
Attachment #8604828 - Flags: review?(jaws) → review+
(adding to my checkin queue)
Keywords: checkin-needed
Comment on attachment 8604828 [details] [diff] [review]

[Triage Comment]

a+ for aurora/beta/release: required for Pocket launch in 38.0.5.
Attachment #8604828 - Flags: approval-mozilla-release+
Attachment #8604828 - Flags: approval-mozilla-beta+
Attachment #8604828 - Flags: approval-mozilla-aurora+
Assignee: nobody → nate
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
You need to log in before you can comment on or make changes to this bug.