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

Integrate new Storybook 5.3+ config & docs add-on #435

Merged
merged 3 commits into from
Jan 21, 2020
Merged

Integrate new Storybook 5.3+ config & docs add-on #435

merged 3 commits into from
Jan 21, 2020

Conversation

kylemh
Copy link
Contributor

@kylemh kylemh commented Jan 20, 2020

Resolves #430
Resolves #431

I noticed in #318 Swyx asked for the docs addon, so I went ahead and integrated it.

Resolving 430 is crystal clear, but 431 I didn't see why it was ever implemented in the first place and have simply taken it out.

Copy link
Collaborator

@swyxio swyxio left a comment

Choose a reason for hiding this comment

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

sick! you rock kyle. idk if @jaredpalmer or @agilgur5 have any other opinions but otherwise this LGTM

@jaredpalmer
Copy link
Owner

Merge it

@jaredpalmer jaredpalmer merged commit 1d8f123 into jaredpalmer:master Jan 21, 2020
@kylemh kylemh deleted the integrate-new-storybook-config branch January 21, 2020 22:41
@swyxio
Copy link
Collaborator

swyxio commented Jan 24, 2020

thanks for the great PR! @all-contributors please add @kylemh for code, examples and tests

@allcontributors
Copy link
Contributor

@sw-yx

I've put up a pull request to add @kylemh! 🎉

@agilgur5 agilgur5 changed the title Integrate new Storybook config Integrate new Storybook 5.3+ config & docs add-on May 3, 2020
@agilgur5 agilgur5 added the scope: templates Related to an init template, not necessarily to core (but could influence core) label Sep 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: templates Related to an init template, not necessarily to core (but could influence core)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot find module '@' with Storybook template Use new Storybook 5.3+ config file
4 participants