Closed misama-ct closed 1 month ago
The latest updates on your projects. Learn more about Vercel for Git βοΈ
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
ui-kit | β Ready (Inspect) | Visit Preview | π¬ Add feedback | Aug 7, 2024 0:34am |
ui-kit-storybook-v8 | β Ready (Inspect) | Visit Preview | π¬ Add feedback | Aug 7, 2024 0:34am |
Latest commit: 82397b5a210c3bb7f33626d686b096b163859b48
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
This pull-request aims to provide an alternative story in typescript format for (almost) every existing story in the
packages/components
folder. The stories are added to thesrc
folder of each component and are accompanied by areadme.mdx
file that displays the auto-generated README.md file of the component within storybook.The
*.stories.tsx
files are no engineering highlight (e.g. I didn't fix most type-errors), but they will do for now, as the current goal is to aim for feature-parity between the existing stories and the new typescript sb v8 stories. We want engineers to have the same or better DX when accessing storybook.Shortcuts taken/compromises made
*.stories.tsx
are excempt from linting and type-checking for now and may or may not contain typescript errors caused by the story or the component being demo'd. The reason is, that all imports to @storybook/react (have to) resolve to the v5 version until the v5 version of storybook is removed.*.form.story.js
stories. They were basically explaining the usage of formik over and over and contained a lot of boilerplate code only useful in the storybook context (a maintenance burden essentially).Suggested focus for the review
Props
page should show all props + their respective inputs for the particular componentBasic
story should either be equivalent to the legacy story, or demonstrate the simplest basic usage followed by more stories (demonstrating more complex use cases)Next steps (happy path)
Just for context, here are the next issues/tasks I'm gonna tackle once this pull-request is merged: