Skip to content

Fix: Storybook setup#1033

Merged
mohdsayed merged 7 commits into
developfrom
fix/storybook-setup
Apr 16, 2025
Merged

Fix: Storybook setup#1033
mohdsayed merged 7 commits into
developfrom
fix/storybook-setup

Conversation

@CharliePops

@CharliePops CharliePops commented Apr 4, 2025

Copy link
Copy Markdown
Contributor

Description

Fix Storybook and make sure there are no vulnerabilites with dependencies

Testing Instructions

  1. Pull and checkout the branch fix/storybook-setup
  2. Build and run the extension: npm install && npm run storybook
  3. Observe: no vulnerabilites are detected by npm and storybook run properly

Checklist

  • I have thoroughly tested this code to the best of my abilities.
  • I have reviewed the code myself before requesting a review.
  • This code is covered by unit tests to verify that it works as intended.
  • The QA of this PR is done by a member of the QA team (to be checked by QA).

@CharliePops CharliePops self-assigned this Apr 4, 2025
@CharliePops CharliePops force-pushed the fix/storybook-setup branch from 801c5fb to 86bafa8 Compare April 4, 2025 15:41
@CharliePops CharliePops requested a review from mohdsayed April 7, 2025 09:16
Comment thread package.json Outdated
Comment on lines +165 to +167
"dependencies": {
"npm": "^11.2.0",
"run": "^1.5.0"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the purpose of it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the storybook installer added them, but I am not fully sure why, I will check if this is really needed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, not really sure why this is there, probably a typo while installing deps. Anyways, I removed it and works fine.

@mohdsayed mohdsayed modified the milestones: v1.0, Future Releases Apr 16, 2025
@mohdsayed mohdsayed merged commit 928ce97 into develop Apr 16, 2025
@mohdsayed mohdsayed deleted the fix/storybook-setup branch April 16, 2025 12:30
@mohdsayed mohdsayed mentioned this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants