-
Notifications
You must be signed in to change notification settings - Fork 524
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
[bug] react version dependencies locked at 16.x #393
Comments
Hi @amcdnl , first off, just making sure you have at least npm 7 for the peer dependency part. Yes we ought to be able to support React@17, as long as |
|
Hi Austin, as we rely on @fluentui/react version 7, we are in lockstep with their peer deps. See microsoft/fluentui#20145 Their version 8 however, specifies compat with React 17. Which means to support this at this time, we'd need to major-version-bump our packages:
Which means supporting current v3 and next v4 of each of these packages. It may be easier for you to republish under your own org on npm (swap out @msrvida with your org name) if you're open to that. |
My current plan is to major bump all packages in this repo to v4 (in a future major release) and at that time, set React 17 as the minimum. |
@danmarshall - Thanks for the update. Is there a timeframe for that set yet? |
@amcdnl Nothing in stone, but shooting for mid-year. |
Thanks @danmarshall - 18 will be out by then :D |
Hi @amcdnl , we've got a beta of our next major version 4 which uses react@17. I know that 18 is now out, but @fluentui/react is |
Thanks for the update! We've moved to 18 already but happy to have it closer. |
Issue
When trying to install sanddance-react I get the following error:
My dependencies are set to:
I believe the package should loosen these dependencies given that 17.x has no breaking changes for this project.
The text was updated successfully, but these errors were encountered: