-
Notifications
You must be signed in to change notification settings - Fork 4
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
Sanchit comments #47
Comments
@sanchit94 if you wish - we can discuss here a tasks, that should be done by other React interns, in order to make code of this repository more clear... |
Hey Arthur, I think a few things can be done to make this repo's code better:
|
Yeah, i think your code is a good starting point, but it should be
improved, because we're trying to put more functionality into it.
Thank you for our comments. Tell me if you find something else
Btw, i also think that we should start to use forms from your #23
repository - because code looks good at that files...
|
Yeah react-final-form code is really great, I think it should be used for the form part. |
I want to continue with this project on react but due to shortage of time I am not able to get my head into it. Additionally I am working on project in Angular so I have much better perspective of these frameworks now. However, once I am done with the Angular project I would love to contribute to this project as I have not written JAVASCRIPT for a while! |
cool! this is a perfect news! we actually re-forming our powers and starting to work on main GS React frontend |
GroceriStar/react-only-intern-23#70 (comment)
The text was updated successfully, but these errors were encountered: