-
Notifications
You must be signed in to change notification settings - Fork 6
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
Upgrade to Helix v0.19, React 16, and Storybook #77
Conversation
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.
Any thoughts about updating all the dependencies?
Deploy preview for helix-react processing. Building with commit c98edb5 https://app.netlify.com/sites/helix-react/deploys/5e77dc764fee5df20a560e90 |
Deploy preview for helix-react ready! Built with commit a479c7d |
- Most of the storybook setOptions have been deprecated, decided to remove them in favor of new defaults storybook 5. (Name and URL are now set as part of the theme) - webpack.config.js no longer works in new version of storybook. Opting for defaults instead.
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.
Looks awesome @nicko-winner...thanks for updating all of the dependencies!
Just one small change.
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.
Dev LGTM
Upgrade Helix to v0.19, React 16, and Storybook 5
Storybook 3 (Before):

Storybook 5 (After):

todo: