If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Remove Services.obs code that we don't use right now

RESOLVED FIXED

Status

L20n
JS Library
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: stas, Assigned: stas)

Tracking

Details

(Whiteboard: [gecko-l20n])

Attachments

(1 attachment)

40 bytes, text/x-github-pull-request
gandalf
: review+
Details | Review | Splinter Review
(Assignee)

Description

a year ago
l20n-chrome-xul and l20n-chrome-html both have code that requires additional dependencies to land in Gecko and that cannot be currently tested.  It's the code that we used for the interactive localization demo in London.

The solutions from that demo are easily available for us to see in the l20n branch of gecko-dev on github.  Let's remove this code from master and re-add it when we actually need it.
(Assignee)

Updated

a year ago
Assignee: nobody → stas
Status: NEW → ASSIGNED
Whiteboard: [gecko-l20n]
(Assignee)

Comment 1

a year ago
Created attachment 8799379 [details] [review]
Pull request
Attachment #8799379 - Flags: review?(gandalf)
Comment on attachment 8799379 [details] [review]
Pull request

r+!
Attachment #8799379 - Flags: review?(gandalf) → review+
(Assignee)

Comment 3

a year ago
Thanks!

https://github.com/l20n/l20n.js/commit/eaf945eec8cdf94a29820ed30f20606ed70f86fa
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
(Assignee)

Comment 4

a year ago
https://hg.mozilla.org/projects/larch/rev/ec2da84091f9ec100e6a16b9e56ecca95404e479
Bug 1308896 - Remove unused language-registry-* observers. r=gandalf
You need to log in before you can comment on or make changes to this bug.