-
Notifications
You must be signed in to change notification settings - Fork 64
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: convert to react@18 only #1044
feat: convert to react@18 only #1044
Conversation
BREAKING CHANGE: Requires `react@18` or higher.
BREAKING CHANGE: Requires `react@18` or higher.
BREAKING CHANGE: Requires `react@18` or higher.
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.
Has there been any discussion in replacing react-beautiful-dnd
?
<UIDReset> | ||
<UIDFork> |
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.
💭
We have had chats about every so often offline, but we should have a better discussion here on GitHub. |
What?
Converts BigDesign to be
react@18
only.ℹ️ Reviewing by commit will be helpful!
BREAKING CHANGE:
Requires
react@18
or higher.Why?
So we can get the benefits of SSR react using the new
useId
hooks.Screenshots/Screen Recordings
N/A
Testing/Proof
react-beautiful-dnd
doesn't work with strict mode.