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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

update storybook example (copy from vercel/vercel) #50865

Merged
merged 3 commits into from Jun 14, 2023

Conversation

Ethan-Arrowood
Copy link
Contributor

Updates the with-storybook example to be the example located in vercel/vercel.

The motivation for this change is that we now have better support for storybook. It is now treated as its own framework and so users can specify the storybook framework as well as custom build command build-storybook in order to get near zero-config support 馃専

@ijjk ijjk added examples Issue/PR related to examples type: next labels Jun 6, 2023
@Ethan-Arrowood
Copy link
Contributor Author

Whoops I think i started from main. Forgot y'all use canary. Let me fix

Copy link
Member

@ijjk ijjk left a comment

Choose a reason for hiding this comment

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

If we're wanting to keep these examples in sync we should probably link one to other so there's a single source of truth

@Ethan-Arrowood
Copy link
Contributor Author

Which should be source? I like that you can use this with create-next-app. So we should make this one the source?

@ijjk
Copy link
Member

ijjk commented Jun 6, 2023

Allow CI Workflow Run

  • approve CI run for commit: 49cd139

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@ijjk
Copy link
Member

ijjk commented Jun 14, 2023

This one being source of truth sounds good to me

ijjk
ijjk previously approved these changes Jun 14, 2023
@kodiakhq kodiakhq bot merged commit d31ced1 into vercel:canary Jun 14, 2023
32 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue/PR related to examples locked type: next
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants