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

Vue Guide for Storybook v6 #405

Merged
merged 19 commits into from
Nov 20, 2020
Merged

Conversation

andre-brdoch
Copy link
Contributor

Hey! I updated the Vue guide to storybook v6 😃

Snippets can be tested here: https://github.com/andre-brdoch/learnstorybook-v6-vue
@jonniebigodes already reviewed the snippets there 😄

I noticed that some chapters like the one for testing or deploys were quite different in the React guide. I assumed the React guide is most up to date, so i adapted those for Vue. Let me know if that was not wanted!

I added the snippets only to the english version, since it probably makes more sense to update snippets and texts simultanously for the other languages. Have a good day 👍

Copy link
Collaborator

@jonniebigodes jonniebigodes left a comment

Choose a reason for hiding this comment

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

@andre-brdoch left two items that need some work. Other than that we're good to go. Can't thank you enough for this. Really stoked to get this one merged and shared with the community. Awesome job you did here :)

content/intro-to-storybook/vue/en/composite-component.md Outdated Show resolved Hide resolved
content/intro-to-storybook/vue/en/data.md Outdated Show resolved Hide resolved
@andre-brdoch
Copy link
Contributor Author

Hi @jonniebigodes, thank you so much for pointing this out! Totally missed that I was not actually running the tests 🙃 Thanks for letting me know, this is great to know for the future. I applied the fixes, let me know if there is anything left to do 😃

Have a good one!

Copy link
Collaborator

@jonniebigodes jonniebigodes left a comment

Choose a reason for hiding this comment

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

@andre-brdoch when you're able address the two items i pointed out regarding the name of the branches and we'll be good to go.

Let me know once you've applied them and i'll get this merged.

Stay safe

content/intro-to-storybook/vue/en/deploy.md Outdated Show resolved Hide resolved
content/intro-to-storybook/vue/en/deploy.md Outdated Show resolved Hide resolved
@andre-brdoch
Copy link
Contributor Author

@jonniebigodes done! 👍

@jonniebigodes
Copy link
Collaborator

@andre-brdoch going to merge it as is. Thank you for this much deserved update to the tutorial. We really appreciate it. 👍

Stay safe.

@jonniebigodes jonniebigodes merged commit 7912339 into chromaui:master Nov 20, 2020
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