Update strings in tree for Pocket

RESOLVED FIXED in Firefox 58

Status

()

Firefox
Pocket
RESOLVED FIXED
2 months ago
a month ago

People

(Reporter: flod, Assigned: mkaply)

Tracking

Trunk
Firefox 59
Points:
---

Firefox Tracking Flags

(firefox58 fixed, firefox59 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

2 months ago
Strings are here
https://github.com/mozilla-l10n/pocket-l10n

Process is documented here
https://github.com/mozilla-partners/pocket

Strings haven't been updated in mozilla-central for a long time. We should also uplift the changeset to Beta.
(Reporter)

Comment 1

2 months ago
@Mike
Do you know who can take this bug?
Flags: needinfo?(mozilla)
Comment hidden (mozreview-request)
(Assignee)

Updated

a month ago
Assignee: nobody → mozilla
Flags: needinfo?(mozilla)

Comment 3

a month ago
mozreview-review
Comment on attachment 8934320 [details]
Bug 1417637 - Update l10n strings for pocket.

https://reviewboard.mozilla.org/r/205244/#review210822

lgtm
Attachment #8934320 - Flags: review?(mixedpuppy) → review+

Comment 4

a month ago
Pushed by mozilla@kaply.com:
https://hg.mozilla.org/integration/autoland/rev/973ee15016b8
Update l10n strings for pocket. r=mixedpuppy

Comment 5

a month ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/973ee15016b8
Status: NEW → RESOLVED
Last Resolved: a month ago
status-firefox59: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
(Reporter)

Comment 6

a month ago
Comment on attachment 8934320 [details]
Bug 1417637 - Update l10n strings for pocket.

Approval Request Comment
[Feature/Bug causing the regression]: -

[User impact if declined]: several languages missing

[Is this code covered by automated tests?]: no

[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]: -

[Is the change risky?]: no

[Why is the change risky/not risky?]: no code changes, only affect secondary languages

[String changes made/needed]: included in the patch
Attachment #8934320 - Flags: approval-mozilla-beta?
Comment on attachment 8934320 [details]
Bug 1417637 - Update l10n strings for pocket.

Update strings for pocket. Beta58+.
Attachment #8934320 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Updated

a month ago
status-firefox58: --- → affected
You need to log in before you can comment on or make changes to this bug.