-
Notifications
You must be signed in to change notification settings - Fork 27
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
Migrate react classes to use the latest hooks #29
Comments
Hi, |
Hey @abhinav5481 , Good luck. Ping me if you need any help. |
I would like to work on this. Just one doubt: Do I need to change every file with class components to hooks or some specific files only? |
Hello @shmiitg , Thank you for your interest in this - we have to migrate every class from classes to hooks but we could do it one by one. Starting from non impacted defaultapps like say minesweeper etc |
Ok sir, I got it. I will start working on this. You can assign me this issue. |
Sir i have changed the following files: I am sending a pr, if you dont find any mistakes you can merge it, I will slowly and gradually change all the files. |
Either migrating one class at a time(All the essential window wrappers and then the pre installed applications) or Migrating it all at once, Whichever method works fine.
The text was updated successfully, but these errors were encountered: