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

[manager] Change API to disable composition #12326

Closed
wants to merge 2 commits into from

Conversation

kylegach
Copy link
Contributor

@kylegach kylegach commented Aug 31, 2020

Issue: API to disable composition of a Storybook is incorrect, as it uses disabled instead of disable.

What I did

refs[key].disabled -> refs[key].disable

How to test

Try to disable composition. Not sure if this needs tests, or how to test it.

@shilman
Copy link
Member

shilman commented Aug 31, 2020

@ndelangen if this is indeed a typo we should probably fix the feature to accept disable since that's what we use in the rest of the Storybook API

- `refs[key].disabled` -> `refs[key].disable`
@kylegach
Copy link
Contributor Author

kylegach commented Aug 31, 2020

Thanks, @shilman. That's a much better way forward. I went ahead and pushed a commit to the branch that reverts the prior commit and changes the code in question.

@kylegach kylegach changed the title [docs] Fix typo in Package Composition page [manager] Change API to disable composition Aug 31, 2020
@kylegach
Copy link
Contributor Author

Closing because this was superseded by #12603.

@kylegach kylegach closed this Sep 30, 2020
@kylegach kylegach deleted the patch-1 branch September 30, 2020 13:52
@shilman
Copy link
Member

shilman commented Oct 1, 2020

Thanks @kylegach! So sorry I missed this ... sometimes it's like drinking from a firehose 😭

@kylegach
Copy link
Contributor Author

kylegach commented Oct 1, 2020

No worries! It happens. 🙃

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.

None yet

3 participants