-
Notifications
You must be signed in to change notification settings - Fork 5
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
#349: update stories to typescript #373
Conversation
…349_storybook-typescript Conflicts: package.json src/plugins/v-focus/styles/styles.scss src/stories/elements/e-button.stories.ts src/stories/elements/e-checkbox.stories.ts src/stories/elements/e-video.stories.ts
"fix:eslint": "npm run lint:eslint -- --config .eslintrc.fix.cjs --cache=false --fix", | ||
"fix:stylelint": "npm run lint:stylelint -- --cache=false --config .stylelintrc.fix.js --fix", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question: are these triggered in the pre-commit hook or just here for manual execution?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are for manual execution
render: (args) => ({ | ||
const Template: StoryObj = { | ||
// eslint-disable-next-line @typescript-eslint/no-explicit-any | ||
render: (args: any) => ({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note: I don't like having any here. But I think for now it is sufficient enough.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't get it to work without it. Maybe I missed something here.
Pull request
Refactors storybook stories to typescript.
Ticket
#349
Browser testing
Checklist
Review/Test checklist