-
Notifications
You must be signed in to change notification settings - Fork 408
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
React 17 support #713
Comments
Looks like there is already a PR here: #681 |
+1 for merging the PR, plz :) |
Look forward to supporting React17 soon |
+1 for merging this PR |
+1 for merging the PR, plz :) |
+1 for merging the PR! |
+100 |
FWIW this library as it stands does work with react 17, you just get a peer dependency warning that you can ignore (at least with yarn) |
+1 |
So far as I know with NPM 7, you can either ignore all warnings or none. It's such a trivial thing to just make a new release with the peer dependencies versions updated. |
@zachsa completely agreed, just trying to help unblock folks. |
+1 |
1 similar comment
+1 |
@tomkp Any updates? |
please update for react@17.. |
Could you please update for React 17? |
@tomkp any update on this? |
I've installed it using the |
+1 |
These are the only changes that the pull request #681 includes - So... I just did
I'm using FluentUI components and the splitter bar doesn't show up or work, but it does split the screen. Which I might as well do with css...
YMMV |
hello, any idea when react 17 will be supported? new version is 17.0.2 |
Currently, the peer dependencies of React are specified to be 16.0.0. Can we support React 17?
The text was updated successfully, but these errors were encountered: