-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Chakra UI Updated from v1.0.0 to v2.10.7 to support after react18^ #4944
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
base: master
Are you sure you want to change the base?
Chakra UI Updated from v1.0.0 to v2.10.7 to support after react18^ #4944
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
✅ Deploy Preview for redux-starter-kit-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 9ea5084:
|
|
LGTM |
…o Sriram-Example-Fix-3
examples/query/react/authentication/src/features/auth/Login.tsx
Outdated
Show resolved
Hide resolved
aryaemami59
left a comment
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.
This resolves the issue. Thanks for the hard work! I’ll wait to merge until we get another set of eyes on it.
Co-authored-by: Arya Emami <aryaemami59@yahoo.com>
This is my first PR in Redux , so please let me know if I missed anything!
🛠 Fix: Chakra UI version upgrade for React 19 compatibility in examples
Chakra UI v1 only supports React up to v17. Since the project now uses React 19, some examples using Chakra UI v1 no longer work correctly in CodeSandbox
To fix this, I’ve updated the Chakra UI version from v1 to v2 in the affected examples to ensure they work with React 19.
Here are a couple of examples where the issue was observed:
Authentication Example
Prefetching Example
Let me know if anything else needs to be updated. Happy to make further changes!