Panel dictionary file missing entries for some languages

RESOLVED FIXED in Firefox 38.0.5



4 years ago
4 years ago


(Reporter: nate, Assigned: nate)


Firefox 41

Firefox Tracking Flags

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



(1 attachment)



4 years ago
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
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
You need to log in before you can comment on or make changes to this bug.