Closed
Bug 1817460
Opened 2 years ago
Closed 2 years ago
Push service should support ES modules
Categories
(Core :: DOM: Push Subscriptions, task)
Core
DOM: Push Subscriptions
Tracking
()
RESOLVED
FIXED
112 Branch
Tracking | Status | |
---|---|---|
firefox112 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
(Whiteboard: [esmification-timeline])
Attachments
(1 file)
In bug 1816934 I am converting services/settings to ES modules. I noticed that the push service only supports legacy JSM modules.
Whilst we can still use the old format name to reference the modules, we should migrate the code to use importESModules
.
In bug 1816934, I am going to hard-code the existing URI so that it will keep working, it should be replaced by import.meta.url
as part of this transition (the approximate location of the code is here).
Assignee | ||
Comment 1•2 years ago
|
||
Updated•2 years ago
|
Assignee: nobody → standard8
Status: NEW → ASSIGNED
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6d8732b3ddd0
Change PushBroadcastService to handle its listeners calling directly into ES modules. r=mt
Comment 3•2 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox112:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•