Skip to content

fix typings: Add children prop - #169

Open
reyronald wants to merge 2 commits into
screener-io:masterfrom
reyronald:patch-1
Open

fix typings: Add children prop#169
reyronald wants to merge 2 commits into
screener-io:masterfrom
reyronald:patch-1

Conversation

@reyronald

Copy link
Copy Markdown

In React 18, the React.Component type no longer includes the children prop by default, so it needs to be included explicitly.

See https://reactjs.org/blog/2022/03/08/react-18-upgrade-guide.html#updates-to-typescript-definitions

Without this change the Screener component cannot be used in React 18 without TypeScript compilation errors.

In React 18, the `React.Component` type no longer includes the `children` prop by default, so it needs to be included explicitly.

See https://reactjs.org/blog/2022/03/08/react-18-upgrade-guide.html#updates-to-typescript-definitions

Without this change the `Screener` component cannot be used in React 18 without TypeScript compilation errors.
@sauce-gabriel

Copy link
Copy Markdown
Collaborator

Hello @reyronald, thank you for reporting the issue along with the fix!
Will be looking into that and report back here as soon as we have an update on it.
Appreciate your collaboration on screener-storybook 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants