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

Update storybook in examples #118

Merged
merged 3 commits into from Oct 26, 2021
Merged

Update storybook in examples #118

merged 3 commits into from Oct 26, 2021

Conversation

IanVS
Copy link
Member

@IanVS IanVS commented Oct 19, 2021

This updates our examples to use storybook 6.4.0-beta.14.

One regression I've noticed so far is that the stories are no longer sorted by the export order, but rather alphabetically. I haven't traced that down to a particular storybook version yet though.

This also uses yarn's workspace: protocol so we don't need to update them every time we bump the version of the builder. (https://yarnpkg.com/features/workspaces#publishing-workspaces)

Copy link
Collaborator

@eirslett eirslett left a comment

Choose a reason for hiding this comment

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

Looks good!

@shilman
Copy link
Member

shilman commented Oct 24, 2021

We've been fixing bugs right and left in the beta, so I'd recommend updating again. I expect things to stabilize this week 🤞

@IanVS
Copy link
Member Author

IanVS commented Oct 24, 2021

I've updated to the latest, but storybookjs/storybook#15574 still seems to be a problem, and the stories are sorted alphabetically.

On my own project, I've noticed that the stories in the sidebar start out in the correct order, but then once the preview pane actually renders in, they re-organize into alphabetical order. I'm not sure why.

@shilman
Copy link
Member

shilman commented Oct 24, 2021

I'll look at it this week

@IanVS
Copy link
Member Author

IanVS commented Oct 26, 2021

Going to land this one, and we can upgrade again once there's a fix out.

@IanVS IanVS merged commit 450dbf3 into main Oct 26, 2021
@IanVS IanVS deleted the update-example-storybook-version branch October 26, 2021 13:16
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