Skip to content
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

Fixed Storybook #5783

Merged
merged 1 commit into from
Jan 28, 2020
Merged

Conversation

nickytonline
Copy link
Contributor

@nickytonline nickytonline commented Jan 28, 2020

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

We'll be using Storybook for the design system, so I've resuscitated it. We're currently using a downgraded version (3.4.11) because we are currently using webpacker 3. Latest versions of Storybook have been on webpack 4 for quite some time now.

There is a bigger task of upgrading webpacker, so for now we will stay with this downgraded version of Storybook. See this Tweet thread.

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

[optional] What gif best describes this PR or how it makes you feel?

Nick Miller from New Girl saying he's a great story teller

@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label Jan 28, 2020
"@storybook/addon-links": "^5.1.9",
"@storybook/addons": "^5.1.8",
"@storybook/react": "^3.4.11",
"@storybook/addon-actions": "3.4.11",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR looks like nothing... it is. However, I was scratching my head for a whole day why downgrading didn't work. I finally literally copied the entire package.json version from the last working Storybook in the repo and it still wasn't working. And then it clicked. I've had this issue before. I had to delete the yarn.lock file and completely regenerate it, to have the proper deps in the lock file. 🙃

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what a doozy

@nickytonline nickytonline requested review from rhymes and jacobherrington and removed request for rhymes January 28, 2020 04:21
@nickytonline
Copy link
Contributor Author

@maestromac @Zhao-Andy, no rush on this, but this means at some point we can revisit #338.

@nickytonline
Copy link
Contributor Author

nickytonline commented Jan 28, 2020

Although I have not changed any of the code for the application, the yarn.lock file had some severe modifications. I would assume all is good, given the build status, but I will do a smoke test in the AM.

Copy link
Contributor

@rhymes rhymes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's go back in time to go forward in time!

I can't wait for us to switch to webpack 4, it'll unlock many superpowers (babel 7, new storybook, various packages that can be upgraded and last but not least, it'll get us closer to Rails 6 which requires webpack 4).

@pr-triage pr-triage bot added PR: reviewed-approved bot applied label for PR's where reviewer approves changes and removed PR: unreviewed bot applied label for PR's with no review labels Jan 28, 2020
@nickytonline
Copy link
Contributor Author

I've done a smoke test and things appear to be good in the app.

@maestromac maestromac merged commit cd25f9a into master Jan 28, 2020
@pr-triage pr-triage bot added PR: merged bot applied label for PR's that are merged and removed PR: reviewed-approved bot applied label for PR's where reviewer approves changes labels Jan 28, 2020
@maestromac maestromac deleted the nickytonline/storybook-downgrade-to-webpack-3 branch January 28, 2020 21:37
@nickytonline nickytonline mentioned this pull request Jan 30, 2020
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged bot applied label for PR's that are merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants