-
Notifications
You must be signed in to change notification settings - Fork 29
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
feat: upgrade chakra from RC to stable release #131
Conversation
@@ -42,8 +42,8 @@ | |||
}, | |||
"dependencies": { | |||
"@apollo/client": "^3.0.2", | |||
"@chakra-ui/core": "^1.0.0-rc.5", | |||
"@chakra-ui/theme": "^1.0.0-rc.5", | |||
"@chakra-ui/react": "1.3.4", |
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.
Latest version has build error on @chakra/popper dependency that this version does not have
@@ -53,7 +53,9 @@ | |||
"apollo-server-express": "^2.18.1", | |||
<% } -%> | |||
"bcryptjs": "^2.4.3", | |||
"cross-fetch": "^3.0.5", | |||
"cross-fetch": "3.0.5", |
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.
Latest version of cross-fetch creates an issue in the apollo client configuration with the headers.
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.
Tested, looks ❤️ !
* feat:upgrade chakra from RC to stable release * lock cross-fetch version * lock version of chakra
This PR updates Bison to use the latest stable release of chakra-ui from the release candidate version.
All components & templates are updated to reflect the library updates.
Changes
Screenshots
(prefer animated gif)
Checklist
Fixes #xxx