-
Notifications
You must be signed in to change notification settings - Fork 19
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
[#349] chore: update storybook #351
base: master
Are you sure you want to change the base?
Conversation
7a0ca7e
to
bea5986
Compare
Indeed, I missed the step where is necessary to reinstall the yarn packages. Updated the description to include |
I tried what you suggested, and here I got the same issue. |
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.
are you able to run can you try running |
What error are you getting? |
@Klakurka should be fixed now, microbundle had to be updated too |
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.
Related to #349
Description
Fixes #349 by updating storybook and addressing the migrations requirements posted at the migration guide.
Test plan
Make sure the storybook runs normally. For that:
react/
folderi.
docker container stop client-paybutton-react-1
ii.
docker container rm client-paybutton-react-1
iii.
docker image rm client-paybutton-react
docker-compose up
and access the storybook in the browser, should be working.