-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
React Hooks Update #2857
Comments
@danieloprado That sounds like a great plan. We've been discussing the move to Hooks for a while now and the timing of your PR couldn't have been better! |
Also fix #2832 |
@sankhadeeproy007 Thank you! If you move on with this idea I'm glad to help you guys to update the code. |
Hi @danieloprado Is this library support React Hooks? It seems not according to the doc? e.g. https://docs.nativebase.io/Components.html#button-def-headref |
@Sampath-Lokuge @akhil-geekyants Is this PR not being merged due to https://github.com/GeekyAnts/nativebase-v3 |
Hi. What about the migration to React hook of NativeBase? Want to be part of contributor if is possible. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
I have gone through these following points
Issue Description
It's is a proposal, we could migrate to use the new React Hooks Api, it's totally compatible with the current code and it can run side by side, I've created a pull request to demonstrate how simple it is and we could do it component by component with no rush.
Reasons:
The text was updated successfully, but these errors were encountered: