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)

task

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.

Blocks: 1949588

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.

Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED

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
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
Blocks: 1953578
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: