-
Notifications
You must be signed in to change notification settings - Fork 734
Commons - exporting props #1002
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
Conversation
…ifiersProps (to extract sub-modifiers props, like MarginModifiers).
@@ -27,5 +27,14 @@ module.exports = { | |||
}, | |||
get withScrollReached() { | |||
return require('./withScrollReached').default; | |||
}, |
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.
This one is not good..
There's a new temp file commons/new.ts
(that will replace commons/index.js eventually)
The typings are already exported there..
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.
Where is the call for the commons/new.ts
file? I see only commons/index.js
...
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.
We're using it in the components that need these typings..
View, Text, Button.. almost all of them..
get BaseComponentInjectedProps() { | ||
return require('./commons').BaseComponentInjectedProps; | ||
}, | ||
get ForwardRefInjectedProps() { | ||
return require('./commons').ForwardRefInjectedProps; | ||
}, | ||
get ModifiersProps() { | ||
return require('./commons').default; | ||
}, |
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.
You need to export the typings in the ts files
generatedTypes/index.d.ts
src/index.ts
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.
Sorry, I don't understand what is what with all the fake files... Why they should be exported in generatedTypes/index
if these are TS fils? And where all the other commons exports in src/index.ts
?
#863 Description
Exporting BaseComponentInjectedProps, ForwardRefInjectedProps and ModifiersProps (to extract sub-modifiers props, like MarginModifiers).
Changelog
Commons - exporting props.