-
Notifications
You must be signed in to change notification settings - Fork 49
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
feat: update react & react-dom to v17 #882
feat: update react & react-dom to v17 #882
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #882 +/- ##
==========================================
+ Coverage 67.06% 67.09% +0.03%
==========================================
Files 128 128
Lines 3206 3206
Branches 934 934
==========================================
+ Hits 2150 2151 +1
+ Misses 1007 1006 -1
Partials 49 49 ☔ View full report in Codecov by Sentry. |
@Ali-D-Akbar could you please merge this as well, we do not have access. |
@@ -162,7 +162,6 @@ exports[`CreateCoursePage renders page correctly with course create error 1`] = | |||
"timeout": 300, | |||
"unmountOnExit": false, | |||
}, | |||
"displayName": "Fade", |
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.
any ideas why this attr has been removed from various snapsots?
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.
Snapshots are updating when we update the @edx/paragon
version from 20.39.3
to 20.40.0.
44606ed
to
c7e647a
Compare
This reverts commit 92095ef.
Ticket
Upgrade React JS to v17
Description
react
&react-dom
to v17, along with@tinymce/tinymce-react
to respective compatible versionsenzyme-adapter-react-16
with@wojtekmaj/enzyme-adapter-react-17
@edx/frontend-platform, @edx/frontend-component-footer-edx