Closed
Bug 1427847
Opened 7 years ago
Closed 7 years ago
Update strings for Firefox 58
Categories
(Firefox :: New Tab Page, defect, P1)
Firefox
New Tab Page
Tracking
()
People
(Reporter: Mardak, Assigned: Mardak)
References
Details
User Story
Attachments
(1 file)
(deleted),
text/x-review-board-request
|
k88hudson
:
review+
gchang
:
approval-mozilla-beta+
|
Details |
Final locale check is January 10th, so we need to get this uplifted before then.
Assignee | ||
Updated•7 years ago
|
Priority: -- → P1
Assignee | ||
Updated•7 years ago
|
Assignee | ||
Comment 1•7 years ago
|
||
For some context, the last export to mozilla-beta was over a month ago:
2017.12.02.1393-6aa2a91e
https://hg.mozilla.org/releases/mozilla-beta/file/tip/browser/extensions/activity-stream/install.rdf.in
Assignee | ||
Comment 2•7 years ago
|
||
gl, gn, si were added since december
Comment hidden (mozreview-request) |
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8939704 [details]
Bug 1427847 - Update strings for Firefox 58.
https://reviewboard.mozilla.org/r/210008/#review216346
Looks good!
Attachment #8939704 -
Flags: review?(khudson) → review+
Assignee | ||
Comment 6•7 years ago
|
||
Comment on attachment 8939704 [details]
Bug 1427847 - Update strings for Firefox 58.
Approval Request Comment
[Feature/Bug causing the regression]: Activity Stream
[User impact if declined]: 73 locales with missing / partially translated strings
[Is this code covered by automated tests?]: Yes
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: Strings only changes
[String changes made/needed]: Yes, these are the translated strings from pontoon.
Attachment #8939704 -
Flags: approval-mozilla-beta?
Comment 7•7 years ago
|
||
(In reply to Ed Lee :Mardak from comment #6)
> [String changes made/needed]: Yes, these are the translated strings from
> pontoon.
Note that this patch doesn't require l10n=XXX in the commit message, it doesn't touch .properties files, and localization happens outside of m-c. It's good to uplift from a l10n point of view.
Comment 8•7 years ago
|
||
Comment on attachment 8939704 [details]
Bug 1427847 - Update strings for Firefox 58.
Update strings of Activity Streams for 58. Beta58+.
Attachment #8939704 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 9•7 years ago
|
||
bugherder uplift |
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•7 years ago
|
Target Milestone: --- → Firefox 58
Assignee | ||
Updated•7 years ago
|
Iteration: 1.26 → 59.4 - Jan 15
Assignee | ||
Updated•7 years ago
|
User Story: (updated)
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•