Panel dictionary file missing entries for some languages

RESOLVED FIXED in Firefox 38.0.5

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: nate, Assigned: nate)

Tracking

unspecified
Firefox 41
Points:
---

Firefox Tracking Flags

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

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Peter noted that we had some conflicts in dictionary.js: https://github.com/Pocket/Firefox/issues/71

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]
dictionaryFixv2.patch

[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+
https://hg.mozilla.org/mozilla-central/rev/08dbb2a63a6b
Assignee: nobody → nate
Status: NEW → RESOLVED
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.