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

Pcss7 storybook test #325

Closed
wants to merge 55 commits into from
Closed

Pcss7 storybook test #325

wants to merge 55 commits into from

Conversation

dalecreativ
Copy link
Contributor

@dalecreativ dalecreativ commented Mar 4, 2021

Storybook and react additions

What we've done:

  • Added react and react-dom
  • Added Storybook react, for react component development
  • Downgraded PostCSS from v8 to v7. See NOTES
  • Updated babel.config.js for react builds
  • Updated the component-template package to help start new react components.
  • Added a component-blockquote, includes a react component, styles and jest tests.

To test:

Start the storybook app

npm ci
npm run storybook

confirm working with stories at localhost:6006

NOTES:
Storybook react 6.1 > 6.2 has had known issues around its dependencies.

This has caused issues with builds for storybook when using packages like postcss v8 in your own projects that use storybook. As a temporary measure, we've downgraded the postcss version to 7, until an official patch has been released.
Issues and discussions from storybook:

UPDATE: Storybook are currently on beta release of 6.2. With some updates and a additional addon @storybook/addon-postcss. We're now able to build without error.
I've set up a branch to help track beta to production release here : https://github.com/coopdigital/coop-frontend/tree/storybook-beta-with-postcss8

…r, followed dir pattern from foundation-forms
…css for browser.Only convert when ran on build using cli
…CI error. Relates to storybook depencies for postcss beign out of date postcss8 and webpack5 causing build failures in storybook. Known issue with storybook that's being worked on
@dalecreativ dalecreativ marked this pull request as ready for review March 8, 2021 09:56
@dalecreativ
Copy link
Contributor Author

I'm going to add a few more updates. Will close this and reopen later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant