-
Notifications
You must be signed in to change notification settings - Fork 77
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
Refractor code to use functional components. #76
Comments
Can I work on this issue? |
@pranay-simejia I don't think this should have a priority currently. But, yeah, please wait for a maintainer to confirm/approve this issue before starting to work on the fix. Thanks 🙂 Cc: @isabelcosta |
@pranay-simejia , @Rahulm2310 is correct. Right now we are trying to get this project to the MVP (minimum viable product) status. You can find more information on documentation on this zulip message. |
Switching to functional components can be done within 2 days. Single person's job. |
Yeah I think there's also a vscode extension to do this too😅 I can't remember its name 🤔 |
i think @jalajcodes you are talking abou this https://marketplace.visualstudio.com/items?itemName=wix.glean |
I have also created this same issue earlier. Here is the issue number #69. |
@shruti1421 @pranay-simejia since we are currently focusing more on completion of MVP(minimum viable product) i don't think this issue is of much use at this moment (considering it will require change in a lot of files and can cause a blocker to other features as well as current PRs) this issue can be put to hold until we have enough features and workforce for managing the project |
ok |
Is your feature request related to a problem? Please describe.
As mentioned in React Docs after react-hook support, one should prefer using functional components.
Describe the solution you'd like
Convert all class based components to functional components.
Describe alternatives you've considered
for example, componentdDidMount can be replaced by useEffect hook.
The text was updated successfully, but these errors were encountered: