-
-
Notifications
You must be signed in to change notification settings - Fork 0
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 Chakra UI 3 #270
Comments
@abachant this could be a little puzzle and a decent chunk of work paying back some tech debt. React could be updated too. |
@petebachant Yeah I'd love to take this on. Should the react update occur in the same PR? |
Might make sense, though it could increase the number of broken things. You could check out the React release notes and try to predict how much additional effort it will add. Might also be smart to create some e2e tests with Playwright first to minimize manual testing. |
Ok, sounds good. I was looking through the existing playwright tests and it seems like there are some errors as it is in regards to login. Would it make sense to do all the e2e tests first in a separate PR? |
I'm not sure how they all work, and none of them are depended on at the moment, so you could even just figure out how to get one working in a PR, and maybe delete the irrelevant ones. |
Guide: https://www.chakra-ui.com/docs/get-started/migration
The text was updated successfully, but these errors were encountered: