-
Notifications
You must be signed in to change notification settings - Fork 119
feat(storybook): working stories #4265
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@@ -1,4 +1,5 @@ | |||
const { addons } = require("@storybook/addons"); | |||
const { addons } = require("@storybook/manager-api"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They deprecated the other package
docs: { | ||
autodocs: false, | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kept this option in as we can set true to create docs. It's pretty cool, creates a prop tables and gives a preview of all storybooks. I disabled though because they could be pretty long depending on how many stories a component has and could potentially hit the chromatic length limit and fail our tests. We can investigate in future but we didn't have it origionally so not required
@@ -281,7 +281,6 @@ export const decorators = [ | |||
|
|||
export const parameters = { | |||
isTestEnvironment, | |||
actions: { argTypesRegex: "^on[A-Z].*" }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deprecated
"@storybook/addon-essentials": "7.6.4", | ||
"@storybook/addon-interactions": "7.6.4", | ||
"@storybook/addon-links": "7.6.4", | ||
"@storybook/addons": "7.6.4", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
depricated
"@storybook/react": "7.6.4", | ||
"@storybook/react-vite": "7.6.4", | ||
"@storybook/test-runner": "0.16.0", | ||
"@storybook/testing-library": "0.2.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
depricated
View your CI Pipeline Execution ↗ for commit 33857c6.
☁️ Nx Cloud last updated this comment at |
|
||
// eslint-disable-next-line import/no-default-export | ||
export default { | ||
title: "Components/Sidebar/FullCompositions", | ||
title: "Components/Sidebar/Docs", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We already had another storybook with the same name and the new version failed to compile with it. This is for the docs site so more appropriately named
Size Change: 0 B Total Size: 1.12 MB ℹ️ View Unchanged
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 33857c6:
|
Paste
|
Project |
Paste
|
Branch Review |
upgrade-react-19-storybook
|
Run status |
|
Run duration | 04m 57s |
Commit |
|
Committer | Kristian Antrobus |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
0
|
|
0
|
|
0
|
|
67
|
View all changes introduced in this branch ↗︎ |
* feat(core): update react version to 19 * feat(core): address TS errors * feat(core): update yarn lock * feat(core): update button tests * feat(core): format * feat(core): import JSX type * feat(core): format * feat(core): update type docs * feat(website): update to next 15 * feat(core): update package json of website * feat(core): run codemod * feat(core): update emotion * feat(core): replace ReactChild with ReactNode * feat(core): fix api path * feat(core): replace css with style * React 19 intersection (#4262) * feat(website): replace react-visibility-sensor with react-intersection-observer * feat(website): fix build * feat(core): fix biome * feat(core): fix tests * feat(core): fix styling * feat(core): update typedocs * feat(website): fix build * feat(button): fix reset button * feat(core): remove react 16 tests * feat(core): update doc * feat(core): update yarn file for tests * feat(core): update testing-library for react 18 tests * feat(core): update yarn file * feat(core): update importd * feat(storybook): working stories (#4265) * feat(storybook): working stories * feat(pr): polish for review * feat(ci): vix inherited versions for CI * fix(theme): fix broken storybook * fix(types): add correct types for react 19 codemod changes (#4266) * fix(types): add correct types for react 19 codemod changes * feat(progress-bar): add interval type --------- Co-authored-by: PixeledCode <ashoaib540918@gmail.com> * feat(core): remove react 16 from peer dependencies * feat(core): add changeset * Update changeset Co-authored-by: Nora Krantz <75342690+nkrantz@users.noreply.github.com> * feat(core): update changeset --------- Co-authored-by: krisantrobus <55083528+krisantrobus@users.noreply.github.com> Co-authored-by: Nora Krantz <75342690+nkrantz@users.noreply.github.com>
Upgraded the version of Storybook to get it running again. required some additional dependency updates