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

Storybook react #341

Merged
merged 77 commits into from
Mar 24, 2021
Merged

Storybook react #341

merged 77 commits into from
Mar 24, 2021

Conversation

dalecreativ
Copy link
Contributor

@dalecreativ dalecreativ commented Mar 24, 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
…components, switch className prop to quoteLarge variant with tests and css
…d packages. Update configs to remove enzyme testSetup
@dalecreativ dalecreativ changed the title WIP: Storybook react Storybook react Mar 24, 2021
@mchadwickweb mchadwickweb merged commit a988e38 into master Mar 24, 2021
@dalecreativ dalecreativ had a problem deploying to staging-coop-design-system March 24, 2021 15:02 Failure
@mchadwickweb mchadwickweb deleted the storybook-react branch August 12, 2021 08:40
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.

2 participants