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

fix(stories): selectStory correct behavior on call with one parameter #5983

Merged
merged 1 commit into from
Mar 9, 2019
Merged

fix(stories): selectStory correct behavior on call with one parameter #5983

merged 1 commit into from
Mar 9, 2019

Conversation

Armanio
Copy link
Member

@Armanio Armanio commented Mar 9, 2019

Issue: #5950

copy from #5953

What I did

Little fix for selectStory function. When in selectStory passed kind name will redirect on first story in needed kind.
For more information, read that issue.

How to test

Run jest tests
Add links addon to you storybook, call linkTo with kind name and enjoy redirect to first story in kind

  • Is this testable with Jest or Chromatic screenshots?
    Yep! I add new jest test (i have doubts on my grammar, i hope anybody correct me)
  • Does this need a new example in the kitchen sink apps?
    No
  • Does this need an update to the documentation?
    No

Sorry for my bad english, i hope you understand me and feel free for correct me. I will not be upset.

when in `selectStory` passed kind name will redirect on first story in needed kind
tests(stories): add test for call `selectStory` with only kind name
@shilman shilman added bug patch:yes Bugfix & documentation PR that need to be picked to main branch labels Mar 9, 2019
@shilman shilman added this to the 5.0.x milestone Mar 9, 2019
@shilman
Copy link
Member

shilman commented Mar 9, 2019

Awesome -- great job!

@shilman shilman merged commit d3ef9d6 into storybookjs:next Mar 9, 2019
@Armanio Armanio deleted the fix/stories branch March 9, 2019 14:20
@shilman shilman added the patch:done Patch/release PRs already cherry-picked to main/release branch label Mar 11, 2019
shilman added a commit that referenced this pull request Mar 11, 2019
fix(stories): `selectStory` correct behavior on call with one parameter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug patch:done Patch/release PRs already cherry-picked to main/release branch patch:yes Bugfix & documentation PR that need to be picked to main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants