-
Notifications
You must be signed in to change notification settings - Fork 16
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
Change React to StrictMode. #131
Comments
@rubenthoms, @anders-kiaer as I have searched for this feature |
Hey @emadbaqeri. Glad to see that you'd like to help out in this project 😃 I'm afraid, but despite the label (which we should probably remove) this is not really a good issue to start with. Why so? This is because starting using strict mode might cause a lot of issues in multiple components which would make this issue become quite large. Moreover, we haven't quite decided yet when to address this change. However, your help would still be a lot appreciated on other tasks. Could you consider to have a look at some other issues? Some good candidates are:
Looking forward to hearing from you! |
Hey @rubenthoms I'm really happy that you respect a newbie developer and you put your time on halping me. It's my honor to work with you cool guys. I'm going to give it a try and do my best for working on those issues. |
In order to improve code quality, all components should be written/changed to conform to React
Strict Mode
(https://reactjs.org/docs/strict-mode.html).The text was updated successfully, but these errors were encountered: