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

feat(examples): update with-storybook to v7 #49863

Closed
wants to merge 1 commit into from

Conversation

kodai3
Copy link

@kodai3 kodai3 commented May 16, 2023

What?

Updating with-storybook example to use Storybook v7

Why?

Storybook v7 is released and it has zero-config Next.js support.

(and I had a chance to create a minimal repo.)

@kodai3 kodai3 requested review from a team, leerob and steven-tey as code owners May 16, 2023 14:40
@ijjk ijjk added the examples Issue/PR related to examples label May 16, 2023
@dvakatsiienko
Copy link
Contributor

Socket Security: Pull Request Alerts is failing, most likely due to updates in package.json. It happened for mee too.
#49997

Copy link
Member

@leerob leerob left a comment

Choose a reason for hiding this comment

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

Could you share screenshots of a video of this working?

@@ -5,4 +5,4 @@ export default {
component: NextjsImages,
}

export const NextjsImagesPage = () => <NextjsImages />
export const NextjsImagesPage = {}
Copy link
Member

Choose a reason for hiding this comment

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

Why are these all removed?

"@storybook/manager-webpack5": "^6.4.9",
"@storybook/react": "^6.4.9",
"babel-loader": "^8.2.3",
"@storybook/addon-actions": "^7.0.12",
Copy link
Member

Choose a reason for hiding this comment

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

Could we make sure these are on latest?

@balazsorban44
Copy link
Member

Thanks, but this has been addressed already in #50865, so closing this PR! 💚

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 16, 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants