"Pair device" string addition

RESOLVED FIXED in Firefox 11

Status

Android Background Services
Android Sync
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: rnewman, Assigned: rnewman)

Tracking

({late-l10n})

unspecified
mozilla12
ARM
Android
late-l10n

Firefox Tracking Flags

(firefox11 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
I'd like to get a small change landed now, so we can get it neatly into Aurora and into the hands of localizers.
(Assignee)

Comment 1

6 years ago
Created attachment 587223 [details] [diff] [review]
Proposed patch. v1

Feel free to throw this at someone else if I picked the wrong reviewer!
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Attachment #587223 - Flags: review?(blassey.bugs)
Attachment #587223 - Flags: review?(blassey.bugs) → review+
(Assignee)

Comment 2

6 years ago
Thanks blassey!

Will flag for Aurora approval once this hits a green m-c.

https://hg.mozilla.org/integration/mozilla-inbound/rev/01d69766026d
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/01d69766026d
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla12
(Assignee)

Comment 4

6 years ago
Comment on attachment 587223 [details] [diff] [review]
Proposed patch. v1

[Approval Request Comment]
Regression caused by (bug #):
* No regression; new Java-only mobile strings.

User impact if declined:
* Delayed l10n time. This has to land *eventually*, so trying for sooner rather than later. No user impact yet.

Testing completed (on m-c, etc.):
* Builds. (Android builds fail if strings aren't a-ok.)

Risk to taking this patch (and alternatives if risky):
* Negligible.
Attachment #587223 - Flags: approval-mozilla-aurora?

Comment 5

6 years ago
Comment on attachment 587223 [details] [diff] [review]
Proposed patch. v1

[Triage Comment]
Mobile string freeze is coming up, so approving for Aurora now that this has baked on m-c for a couple of days.
Attachment #587223 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 6

6 years ago
> Mobile string freeze is coming up, so approving for Aurora now that this has
> baked on m-c for a couple of days.

Thanks Alex.

Can't land this until the first Android Sync drop lands (Bug 709391). If that doesn't happen, I guess it'll go in the *second* Android Sync drop, which will implicitly include the first and this change.

In the holding area it goes! :D

Comment 7

6 years ago
Richard, is this out of hold? And if so, does this still need to land?
Keywords: late-l10n
(Assignee)

Comment 8

6 years ago
(In reply to Axel Hecht [:Pike] from comment #7)
> Richard, is this out of hold? And if so, does this still need to land?

We got approval for Aurora last week, so I landed everything in one go. Thanks for the catch! (I guess I need to spend time putting together some Bugzilla queries for "things that need attention"...)
Blocks: 721271
status-firefox11: --- → fixed
(Assignee)

Comment 9

6 years ago
Verified:

$ diff ~/moz/hg/mozilla-aurora/mobile/android/base/locales/en-US/sync_strings.dtd ~/moz/hg/mozilla-central/mobile/android/base/locales/en-US/sync_strings.dtd
20c20
< <!ENTITY sync.link.advancedsetup.label 'Advanced setup...'>
---
> <!ENTITY sync.link.advancedsetup.label 'Advanced setup…'>

which is Bug 721305.
Component: Android Sync → Android Sync
Product: Mozilla Services → Android Background Services
You need to log in before you can comment on or make changes to this bug.