-
Notifications
You must be signed in to change notification settings - Fork 404
Convert library from Flow to TypeScript #619
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
Comments
Hi @caiquecastro, thanks for the feature request. I don't have plans to convert the library to TS for now. In the meantime, however, you can use the types provided by DefinitelyTyped. |
Hi @ericgio. Thanks for the reply. Yeah.. I'm using the types from DefinitelyTyped for now! |
Does that help with the issues around Label? I've been trying to solve this and found several SO questions w/o answers to this. index.d.ts(204, 5): The expected type comes from property 'labelKey' which is declared here on type 'IntrinsicAttributes & IntrinsicClassAttributes<AsyncTypeahead> & Readonly<AsyncTypeaheadProps> & Readonly<...>' index.d.ts(242, 5): The expected type comes from property 'options' which is declared here on type 'IntrinsicAttributes & IntrinsicClassAttributes<AsyncTypeahead> & Readonly<AsyncTypeaheadProps> & Readonly<...>' |
@nycynik: If you believe there are problems with the types, please start a discussion over in the DefinitelyTyped repo. |
Thanks, will do. |
As of v6.0.0-alpha.1, the library has been re-written in Typescript. |
Is your feature request related to a problem? Please describe.
I'm having issues to integrate this library with Typescript. But I could send a PR if the library is interested.
Describe the solution you'd like
Replace flow by typescript
How is this solution useful to others?
Flow popularity and usage has been dropping. Typescript is more common.
Describe alternatives you've considered
N/A
Additional context
N/A
The text was updated successfully, but these errors were encountered: