-
Notifications
You must be signed in to change notification settings - Fork 398
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
Add TypeScript / Flow typings #119
Comments
FWIW, Typescript also became pretty popular in the React ecosystem once they added JSX support. Plus if the main Firebase library is typed in TS, we'd have to use that anyway. In any event, my understanding is that it's not too hard to convert from TypeScript definitions to Flow (https://github.com/joarwilk/flowgen). |
Yeah I think I will probably start with TypeScript. |
👍 |
howdy! Is this being worked on? help needed? |
We have officially deprecated reactfire, as we no longer believe it is the best solution for Firebase developers looking to integrate with React. Please see the |
We should provide TypeScript and/or Flow typings for consumers of this repo. Does anyone have any preference for either? TypeScript seems more popular overall and is generally what we use with other Firebase repos, but Flow seems more popular in the React ecosystem due to the Facebook connection. Does anyone know how we can provide both in an easy way?
The text was updated successfully, but these errors were encountered: