Skip to content

Conversation

@ndelangen
Copy link
Member

What I did

  • I moved and renamed storybooks
  • changed wording from "React Storybook" to "Storybook"

Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

Minor change unrelated to your change, but on the same line.


```sh
npm install -save-dev storyshots
npm install -save-dev @storybook/addon-storyshots
Copy link
Member

Choose a reason for hiding this comment

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

should be --save-dev?

Copy link
Member Author

Choose a reason for hiding this comment

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

Nice catch, can you approve now?

@usulpro
Copy link
Member

usulpro commented May 17, 2017

So we officialy rename React Storybook to Storybook?

seems logical 👍
the only doubt how will this affect the indexing in web search?
compare react storybook vs storybook

@ndelangen
Copy link
Member Author

It's still the top hit? I see no problem with this.

It's a needed step.

@ndelangen ndelangen merged commit cf4f080 into master May 17, 2017
@ndelangen ndelangen deleted the restructure-storyshots branch May 17, 2017 15:49
@shilman shilman added the misc label May 27, 2017
@ndelangen ndelangen changed the title MOVE & RENAME and storyshots && CHANGE “React Storybook” > “Storybook” Change brand name from “React Storybook” to “Storybook” May 27, 2017
@ghost ghost mentioned this pull request Nov 11, 2017
@nx-cloud
Copy link

nx-cloud bot commented Aug 6, 2025

View your CI Pipeline Execution ↗ for commit d2cdb0a

Command Status Duration Result
nx run-many -t build --parallel=3 ✅ Succeeded 1m 12s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-06 02:05:23 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants