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
Bump @storybook/react major version #25161
Conversation
Fixes some breakage related to dependency imports.
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 it out and upgrade seems to have fixed the issue.
I didn't notice any issues with the stories either.
Thanks 👍
The size of the output for `npm ls --json --long --prod` has grown to around 110MB. For this reason, the output was being truncated at the previous maximum buffer size, which was set at around ~104MB.
The License Check test was failing because the output of the |
Good catch @vcanales Thanks! |
Description
This fixes some breakage related to dependency imports that we can see now on the playground:
Looking at
moment-timezone
theundefined is not an object (evaluating 'root.moment')
error means that for some reason Storybook is building the dependencies for the browser, and not passingwindow
into the context of the function. I think that Storybook fixed this import/build issue as a part of their new major version :)How has this been tested?
I was able to reproduce the error locally, and it's fixed when updating Storybook.
Types of changes
Bump
@storybook/react
, which was actually reporting that an upgrade was available.