-
Notifications
You must be signed in to change notification settings - Fork 46
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
Update to react 19 #48
Comments
I got similar issue today after upgrading to react 19 |
I got it working with this build config changes https://github.com/asotog/fabricjs-react/compare/master...armedi:fabricjs-react:master?expand=1 You can try it with npm i https://github.com/armedi/fabricjs-react/releases/download/v2.0.1/fabricjs-react-2.0.1.tgz |
Hi @jraut, does this solve your problem? |
This looks really good, thanks @armedi for taking it forward. I have not been able to try it out in a comprehensive manner yet. I wish I could, but updating to React 19 is coming "soon" and therefore I could not prioritise this more so far. |
looks good thanks guys, new version published https://codesandbox.io/p/sandbox/flamboyant-wind-ff3x8 |
Hello,
I just updated to react 19 which is right behind the corner. Ran into a problem:
TypeError: Cannot read properties of undefined (reading 'ReactCurrentDispatcher') at <unknown> (path/node_modules/fabricjs-react/dist/fabricjs-react.es.js:318:15) at Or (path/node_modules/fabricjs-react/dist/fabricjs-react.es.js:759:4) at <unknown> (path/node_modules/fabricjs-react/dist/fabricjs-react.es.js:762:74) at <unknown> (path/node_modules/fabricjs-react/dist/fabricjs-react.es.js:763:3)
The text was updated successfully, but these errors were encountered: