-
Notifications
You must be signed in to change notification settings - Fork 359
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
chore: apply react strict mode and upgrade to React 17 [DET-5325] #2279
Conversation
9908322
to
ef4002e
Compare
This is looking good aside for the logviewer issue, we should probably consider landing this after the release just so that we can have people play around with the WebUI for 2 weeks before releasing it. |
👍 Yeah not looking to rush this one, because issues probably will popup up over time as it's a major upgrade. |
57e6e50
to
c9d668c
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.
looks great! 👍
Description
React Strict Mode
is going to help us migrate toReact v17
.React Strict Mode
and fix all errors and warnings.React Strict Mode Reference
Test Plan
npm start
and check to make sure all the pages still worknpm run storybook
and make sure it runs without errorsnpm run build-storybook
and make sure it runs without errorsCommentary (optional)
Checklist
docs/release-notes/
.See Release Note for details.