Closed
Bug 1949486
Opened 9 months ago
Closed 8 months ago
Migrate all React modules used by Reps/JSONViewer to ES Modules
Categories
(DevTools :: General, task)
DevTools
General
Tracking
(firefox138 fixed)
RESOLVED
FIXED
138 Branch
| Tracking | Status | |
|---|---|---|
| firefox138 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
Details
Attachments
(2 files)
As an independent preliminary step toward migrating a few DevTools compoments to ES Modules we need the ES Modules for React modules.
| Assignee | ||
Comment 1•9 months ago
|
||
Land ES Modules for all React modules.
Also tweak the Browser Loader to force loading react loaded via require() to use the ES Modules.
This helps ensure these modules are loaded only once and always as a ES Module.
Otherwise require() statements from debugger frontend and import from Reps may load
distinct instances.
Updated•9 months ago
|
Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED
| Assignee | ||
Comment 2•9 months ago
|
||
The previous changesets migrated these React modules to ES Modules and renamed them.
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0aa892558b43
[devtools] Expose ES Module versions for React modules. r=devtools-reviewers,nchevobbe
https://hg.mozilla.org/integration/autoland/rev/ea33247cf7fd
[devtools] Allow mjs for react modules in DOM Parser r=ckerschb
Comment 4•8 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/0aa892558b43
https://hg.mozilla.org/mozilla-central/rev/ea33247cf7fd
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
status-firefox138:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•