Skip to content

fix: use storyNames in tree if available#697

Merged
dannyhw merged 2 commits intostorybookjs:nextfrom
aniretaketea:fix/useStoryNames
Mar 19, 2025
Merged

fix: use storyNames in tree if available#697
dannyhw merged 2 commits intostorybookjs:nextfrom
aniretaketea:fix/useStoryNames

Conversation

@aniretaketea
Copy link
Copy Markdown
Contributor

@aniretaketea aniretaketea commented Mar 13, 2025

Issue: storyName property of stories is ignored in tree. There is no open issue, but there is a FIXME in code

What I did

Fixed index object generation in prepareStories

How to test

An example story with storyName was added in examples/expo-example under UI/Sidebar/Tree

@aniretaketea aniretaketea requested a review from dannyhw as a code owner March 13, 2025 18:58
@dannyhw
Copy link
Copy Markdown
Member

dannyhw commented Mar 16, 2025

Hey @aniretaketea thanks so for much for your contribution, I haven't had time to test yet but seems good. Will test it out soon but I anticipate no issues with merging this in.

thanks again.

@aniretaketea
Copy link
Copy Markdown
Contributor Author

@dannyhw glad to be of help and give back! We actively use RN storybook in our project, thanks for maintaining and growing it ♥️

@dannyhw dannyhw merged commit ac09973 into storybookjs:next Mar 19, 2025
1 check passed
dannyhw added a commit that referenced this pull request Mar 22, 2025
* fix: use storyNames in tree if available (#697)

Co-authored-by: Daniel Williams <dannyhyunsoowilliams@gmail.com>

* feat: allow storyName and name

* v8.6.2

---------

Co-authored-by: Ekaterina Tarasova <aniretaketea@users.noreply.github.com>
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.

2 participants