Conversation
|
✅ Deploy Preview for hpe-theme-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for hpe-design-icons-grommet ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for unrivaled-bublanina-3a9bae ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
This PR updates the shared/aries-core Storybook setup by adding a Welcome docs page, simplifying Storybook sidebar sorting/collapsing behavior, and aligning Storybook React dependencies/types with the webpack5 framework package.
Changes:
- Added a new
Welcome.mdxpage intended to be the first entry in the Storybook sidebar. - Updated story files to import
Meta/StoryObjtypes from@storybook/react-webpack5and removed@storybook/reactfrom dependencies. - Adjusted Storybook config for story sorting, collapsed sidebar roots, and MDX story discovery.
Reviewed changes
Copilot reviewed 49 out of 50 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| shared/aries-core/src/stories/components/data/Toolbar.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/DataView.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/DataTableGroupBy.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/DataTableColumns.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/DataTable.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/DataSummary.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/DataSort.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/DataSearch.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/DataFilters.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/DataFilter.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/data/Data.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/ToggleGroup.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Tip.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/TextInput.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/TextArea.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Tag.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Tabs.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Spinner.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Skeleton.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/SelectMultiple.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Select.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/RangeInput.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/RadioButtonGroup.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Pagination.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Page.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Notification.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/NameValueList.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Menu.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/MaskedInput.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5 (and adjust commonArgs import). |
| shared/aries-core/src/stories/components/Main.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Layer.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Header.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Grid.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Footer.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/FileInput.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/DateInput.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/CheckBoxGroup.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/CheckBox.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Card.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Button.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Box.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Avatar.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Anchor.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5. |
| shared/aries-core/src/stories/components/Accordion.stories.tsx | Switch Storybook type imports to @storybook/react-webpack5 (and adjust commonArgs import list). |
| shared/aries-core/src/Welcome.mdx | Add new Storybook welcome page content. |
| shared/aries-core/package.json | Remove @storybook/react dependency (webpack5 framework remains). |
| shared/aries-core/.storybook/preview.mjs | Ensure “Welcome” sorts first and simplify sort ordering. |
| shared/aries-core/.storybook/manager.js | Collapse “components” and “patterns” roots by default in the sidebar. |
| shared/aries-core/.storybook/main.mjs | Restrict MDX story glob to ../src/*.mdx to pick up the new Welcome page. |
| pnpm-lock.yaml | Reflect dependency updates (including removal of @storybook/react). |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)
pnpm-lock.yaml:656
eslint-plugin-storybookappears as a direct dependency in the lockfile, but it is not declared in anypackage.jsonin the repo. This will be removed on the nextpnpm install(or indicates an incomplete dependency update). Either add it to the appropriatepackage.jsondevDependencies or regenerate the lockfile without it.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
shared/aries-core/src/stories/components/MaskedInput.stories.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 49 out of 50 changed files in this pull request and generated 2 comments.
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
https://deploy-preview-5880--unrivaled-bublanina-3a9bae.netlify.app/
What does this PR do?
What are the relevant issues?
#5867
Where should the reviewer start?
How should this be manually tested?
Any background context you want to provide?
Screenshots (if appropriate)
Should this PR be mentioned in Design System updates?
Is this change backwards compatible or is it a breaking change?