-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable support for React 18 #625
Enable support for React 18 #625
Conversation
ec19ce1
to
d973bf1
Compare
d973bf1
to
fa7306f
Compare
0954148
to
599dc60
Compare
599dc60
to
4350d61
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code changes look great! I'm holding off on approving until I have a sandbox spun up so I can do a bit of testing.
The sandbox for openedx/frontend-app-authoring#1675 is up and everything I've tested is looking good! https://app.pr-1675-e8373e.sandboxes.opencraft.hosting/authoring/course/course-v1:demo+Course+1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #625 +/- ##
=============================
=============================
☔ View full report in Codecov by Sentry. |
🎉 This PR is included in version 14.3.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 15.0.0-alpha.24 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
The Open edX MFEs are overdue to support React 18 (especially as React 19 is already out now, too). Some efforts have been started to support React 18 in the shared JavaScript libraries, such as Paragon (openedx/paragon#3367, v22 and v23).
By enabling support for React 18 (should be additive-only), we can further enable incremental migration to React 18 within MFEs. Most of the changes are within the example MFE included in this repository for testing.
The React 18 Upgrade Guide was followed.
Additionally:
@svgr/webpack
is also run on.tsx
files.react-test-renderer
in favor of@testing-library/react
within the example MFE app.redux
andreact-redux
peer dependencies optional, as some MFEs opt to not use Redux in favor of React Query.