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

Svelte: Fix Cannot read property '__docgen' of undefined #14383

Merged
merged 1 commit into from
Mar 29, 2021

Conversation

j3rem1e
Copy link
Contributor

@j3rem1e j3rem1e commented Mar 29, 2021

Issue: storybookjs/addon-svelte-csf#15

When a svelte stories doesn't set the component in the default export (or with in addon-svelte-csf), then the stories doesn't show with "Cannot read property '__docgen' of undefined"

What I did

The svelte source decorators was using the component without checking if it's set or not. I have added a null check here.

How to test

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

Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

LGTM!

@shilman shilman changed the title Fix Cannot read property '__docgen' of undefined Svelte: Fix Cannot read property '__docgen' of undefined Mar 29, 2021
@shilman shilman merged commit d2ad09e into storybookjs:next Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants