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.

Sign up for GitHub

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

Jump to bottom

Upgrade Storybook to v7 #281

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

Upgrade Storybook to v7 #281

wants to merge 31 commits into from

Conversation

rushi
Copy link
Member

@rushi rushi commented Nov 6, 2023 •
edited
Loading

  • 🎉 Upgrade to Storybook v7
  • Add tailwindMerge and created cn utility function

Features of Storybook v7

These are the features of v7 that affect our UI Kit:

  1. We can use vite for Storybook so we no longer have to use Webpack. This makes start/stop much faster. This also makes future migration to TS easier.
  2. It uses the new Component Story Format v3 which makes it much easier to write stories. The amount of code we need to write a story has dropped quite a bit
  3. We have more flexibility in hiding & showing certain feature sets per components
  4. Auto-docs functionality means we don't have to manually write out all the props in the stories
  5. Table of contents for the larger docs pages.
  6. Play functions: They allow us to interact with components without needing user interaction. For example fill out forms, click buttons.

rushi and others added 22 commits November 1, 2023 18:07
@rushi
🎉 Upgrade to Storybook v7 & introduce ability to use Vite & TS
a824c11
@rushi
Remove wrong export
c4bc9da
@rushi
Add tailwindMerge & 'cn'
49b541a
@rushi
Restore ES Lint
0a6d9ff
@rushi
Fix build
a1cc1f9
@rushi
Exclude stories from lint
77677ff
@rushi
Merge branch 'xola:master' into sb-7
75f29cd
@rushi
Merge branch 'master' of github.com:xola/ui-kit into sb-7
59b397f 
* 'master' of github.com:xola/ui-kit:
  NPM audit fix
@rushi
Convert stories to CSF-3
9b91f78
@rushi
Re-use size params since we always have the same size
5504c48
@rushi
Add .xo-config
f2ef0e3
@rushi
Update lint:report script
3a1f46a
@rushi
Set tags: ["autodocs"] to generate Docs page
44b8e52
@rushi
Cleanup and update config
2c11c4c
@rushi
Copy tailwind and postcss config into output folder
6296a7c
@rushi
Remove .mjs file
1f0f65f
@rushi
Rename vars for stories to be clearer
dcad056
@rushi
Remove changes for Docs
829edeb
@rushi
Rename helpers
77d2b29
@rushi
Add pseudo-states and figma plugin
3436105
@rushi
Trim down stories
eac966b
@rushi
Add docs
ea5de23
@rushi rushi marked this pull request as ready for review November 13, 2023 14:33
@rushi
Implement "play" function & trim story code
4560a9c
@rushi
Update changes from master
112cf6c
@rushi
Merge branch 'master' of github.com:xola/ui-kit into sb-7
2fbf4fc 
* 'master' of github.com:xola/ui-kit: (33 commits)
  2.1.39
  X2-8166   Correct Currency not Appearing when Viewing the Customer's Reservation in the Purchases Tab (xola#296)
  2.1.38
  2.1.37
  Revert "X2-8166: Correct Currency not Appearing when Viewing the Customer's Reservation in the Purchases Tab" (xola#294)
  2.1.36
  X2-8166: Correct Currency not Appearing when Viewing the Customer's Reservation in the Purchases Tab (xola#292)
  2.1.35
  X2-6577 Tutorial - Fix  (xola#291)
  2.1.34
  X2 7053 feedback widget (xola#266)
  2.1.33
  X2-6907 Setting default as no date for DatePickerPopover [LATEST] (xola#289)
  2.1.32
  Revert "X2-6907 Setting default as no date for DatePickerPopover" (xola#288)
  2.1.31
  2.1.30
  X2-7919 set initial values on datepicker outside click (xola#287)
  fixing startMonth
  fixing endMonth when value is null
  ...
@rushi
Upgrade Storybook to 7.6 and use ES Modules
7a56416
@rushi
Merge branch 'master' into sb-7
6acff89
@rushi
🧹 Lint
40b51b0
@rushi
Fix build file
d3cefe0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rushi