Correct the file path for the Button story

[DaleSeo]

Issue: I happened to notice that the file path for the Button story was incorrect when looking at Storybook for React tutorial.

What I did

Corrected the file path for the Button story.

How to test

• Is this testable with Jest or Chromatic screenshots?
• Does this need a new example in the kitchen sink apps?
• Does this need an update to the documentation?

If your answer is yes to any of these, please make sure to include it in your PR.

[vercel]

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/storybook/monorepo/rjzottv5g
✅ Preview: https://monorepo-git-fork-daleseo-patch-3.storybook.now.sh

[shilman]

Thanks @DaleSeo, good catch! However, the Button stories are actually located at 1-Button.stories.js! Mind updating?

Also, this seems to be a problem across the Welcome component in ALL frameworks. Any chance you might be able to update the others? I'd really appreciate it!

[shilman]

Thanks @DaleSeo, good catch! However, the Button stories are actually located at 1-Button.stories.js! Mind updating?

Also, this seems to be a problem across the Welcome component in ALL frameworks. Any chance you might be able to update the others? I'd really appreciate it!

[DaleSeo]

Oops! Sorry @shamin, you're right. I've updated it!

In terms of updating the others, since I'm not very familiar with this project, I just manually searched for some places and tried to update as much as I could.

I also look for the vue example and found it has the correct path.

storybook/examples/vue-kitchen-sink/src/stories/Welcome.vue

Line 31 in 4e09949

located at <code class="code">src/stories/Button.js</code>.)

[DaleSeo]

I think my changes break some snapshot testing. I guess I need to learn just -u to update the snapshots?

[shilman]

Thanks! Yeah, you should be able to do yarn test --core --update

[DaleSeo]

@shilman I've updated the snapshots but I think chromatic tests are failing now. Could you advise?

[shilman]

Thanks @DaleSeo!! I'll take care of Chromatic & merge. Much appreciated 🙏🙏🙏

[stale]

Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 30 days. Thanks!