update react-with-storybook template for Storybook v6#805
update react-with-storybook template for Storybook v6#805agilgur5 merged 3 commits intojaredpalmer:masterfrom
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/formium/tsdx/4biloi9zk |
agilgur5
added
templates
scope: templates
| /test | ||
| blah.test.tsx # EDIT THIS | ||
| /stories | ||
| Thing.stories.tsx |
There was a problem hiding this comment.
| Thing.stories.tsx | |
| Thing.stories.tsx # EDIT THIS |
| // see: https://github.com/storybookjs/storybook/issues/9556 | ||
| /** | ||
| * A custom Thing component. Neat! | ||
| */ |
There was a problem hiding this comment.
what's the purpose of adding these JSDoc comments?
There was a problem hiding this comment.
To demonstrate the storybook docs generation using these comments.
There was a problem hiding this comment.
gotcha thought that was it -- does that apply to the props as well?
There was a problem hiding this comment.
wow - TIL 😂
how do we comment a comment so it's self-explanatory!?
kylemh
left a comment
kylemh
left a comment
There was a problem hiding this comment.
This should work! Can you please add the global on* parameter for actions to automatically work?
https://github.com/AirLabsTeam/air-core/blob/main/.storybook/preview.js
HipsterBrown
force-pushed
the
update-react-storybook-template
branch
from
9a719ab to
f9d5afc
Compare
agilgur5
left a comment
agilgur5
left a comment
There was a problem hiding this comment.
Thanks for the iteration @HipsterBrown! Few more comments left. Seems like this'll be able to go out with v0.13.3 before I start merging in all the v0.14.0 changes
| /test | ||
| blah.test.tsx # EDIT THIS | ||
| /stories | ||
| Thing.stories.tsx #EDIT THIS |
There was a problem hiding this comment.
| Thing.stories.tsx #EDIT THIS | |
| Thing.stories.tsx # EDIT THIS |
can we make sure this is consistent with the others per my previous suggestion
|
Thanks again, @HipsterBrown. Been swamped with work. |
agilgur5
left a comment
agilgur5
left a comment
There was a problem hiding this comment.
Great work @HipsterBrown !
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
@all-contributors please add @HipsterBrown for code, docs, example |
|
I've put up a pull request to add @HipsterBrown! 🎉 |
|
@all-contributors please add @kylemh for review, questions |
|
I've put up a pull request to add @kylemh! 🎉 |
agilgur5
modified the milestones:
v0.13.x - Template Improvements & More,
v0.14.0 - Dep Upgrades - Jest 25 + Node 10
- remove `@storybook/addon-actions`, `@storybook/addon-docs`, `react-docgen-typescript-loader`, and `ts-loader` - add`@storybook/addon-essentials` - significantly simplify Storybook config - add JS/JSX support - fix a now deprecated globbing pattern - update the example Story to v6 usage - add JSDoc comments to example component to display Storybook docs generation - add a `preview.js` to automatically turn on all actions for event handlers - update/fix file structure in README
3 participants
With the release of Storybook v6, there are updates to be made to migrate from v5.3 and allow the
react-with-storybooktemplate take advantage of the latest features, i.e. built-in Typescript support and new story format.