Skip to content
This repository has been archived by the owner on Nov 11, 2023. It is now read-only.

Fix React import on generated d.ts #95

Merged
merged 1 commit into from
Nov 20, 2018
Merged

Conversation

diegochavez
Copy link
Contributor

React should be imported with * because of React.ReactNode is not being exported with React Object I'm using TSX

Why

After installing the package it throws an error React has no default export
image

React should be imported with * because of React.ReactNode is not being exported with React Object I'm using TSX
@fabien0102
Copy link
Contributor

First of all, good catch, Poll is actually the only file without the * as pattern 😮

But it's really strange, we are using this project in many of our products and never had this error before and allowSyntheticDefaultImports: true in the tsconfig.json should solve this 🤔

Copy link
Contributor

@fabien0102 fabien0102 left a comment

Choose a reason for hiding this comment

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

Since it's the only file not in the project convention, this modification is good to me 💯

@TejasQ
Copy link
Contributor

TejasQ commented Nov 20, 2018

WOW @diegochavez, great work! Thanks for this amazing PR! 🔥

@TejasQ TejasQ merged commit 347207b into contiamo:master Nov 20, 2018
@TejasQ
Copy link
Contributor

TejasQ commented Nov 20, 2018

Published to Restful React v6.1.1!

@diegochavez
Copy link
Contributor Author

Awesome thanks, great library guys! 👍

@TejasQ
Copy link
Contributor

TejasQ commented Nov 20, 2018

💞

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

Successfully merging this pull request may close these issues.

3 participants