-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
docs(applying-storybook): update documentation and usage #12837
docs(applying-storybook): update documentation and usage #12837
Conversation
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
8bb0e32
to
1c9f6ad
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great @TylerAPfledderer, lgtm!
Will wait for @corwintines review in case we are missing something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left one suggestion, but don't think it needs to block merging. Nice one @TylerAPfledderer
As essentially task #0 in rolling out new initiatives in including more component stories, this PR updates and adds clarification to the
Applying Storybook
documentation for contributors.This also updates existing stories to ensure they are inline with this updated documentation.