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

chore: upgrade Storybook to v8.3.6 #1724

Merged
merged 13 commits into from
Nov 7, 2024
Merged

chore: upgrade Storybook to v8.3.6 #1724

merged 13 commits into from
Nov 7, 2024

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Nov 5, 2024

Key features

  1. upgrade Storybook to 8.3.6 so it can run with latest cli-app-scripts and Webpack 5
  2. update the instance endpoint used in some stories
  3. use ConfigProvider in the PeriodDimension stories to avoid crashes

Description

In another PR I had to disable the Storybook build from package.json due to compiling issues.
The reason that happened is the updated version of cli-app-scripts which updates React scripts and thus Webpack.
The old Storybook version is not compatible with Webpack 5.

This PR upgrades Storybook to a recent version which works with Webpack 5, thus allows us to update cli-app-scripts which is required for the new plugin system.


Known issues

  • a version conflict for eslint-plugin-react caused the Stoybook to not build correctly. The root cause was 2 different versions of the eslint dependency. This has been solved by using a resolution for eslint.

Screenshots

Screenshot 2024-11-07 at 10 36 46

@HendrikThePendric HendrikThePendric self-assigned this Nov 6, 2024
@edoardo edoardo marked this pull request as ready for review November 7, 2024 09:30
@edoardo edoardo changed the title chore: upgrade Storybook chore: upgrade Storybook to v8.3.6 Nov 7, 2024
@edoardo edoardo merged commit 0e7b8f3 into master Nov 7, 2024
7 checks passed
@edoardo edoardo deleted the chore/upgrade-storybook branch November 7, 2024 13:19
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 26.9.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants