Skip to content

Setup basic monorepo structure #1

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

Closed
wants to merge 2 commits into from
Closed

Conversation

chrishutchinson
Copy link
Owner

Setup

To run the Storybook:

  1. Clone this fork
  2. Install dependencies
$ yarn # Note: it is important to use Yarn, we use Yarn Workspaces!
  1. Start the Storybook
yarn storybook
  1. The Storybook should open at localhost:9009

Screenshots

Screen Shot 2019-03-11 at 18 28 48

Screen Shot 2019-03-11 at 18 28 42

Screen Shot 2019-03-11 at 18 28 34

Screen Shot 2019-03-11 at 18 28 24

@pietrop
Copy link

pietrop commented Mar 15, 2019

ok, finally found time to have a look a this.

I cloned, cd into folder, run yarn, but when I do yarn storybook I get an error?

$ yarn storybook
yarn run v1.13.0
error Command "storybook" not found.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Do you need storybook installed globally or could be a dev dependency?

@pietrop
Copy link

pietrop commented Mar 15, 2019

My bad, was on master, need to pull feature/monorepo branch 🤦‍♂️

git checkout --track origin/feature/monorepo

or if starting from scratch

git clone -b feature/monorepo [email protected]:chrishutchinson/react-transcript-editor.git

@chrishutchinson
Copy link
Owner Author

Closing because it's since become wildly out of date due to a large UI PR went into bbc/react-transcript-editor@master. I'll be opening a new PR shortly that re-applies the basis of this work on the most up-to-date codebase.

@chrishutchinson chrishutchinson deleted the feature/monorepo branch March 19, 2019 00:53
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