Skip to content
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

feat: Replace class components and states with functional components and hooks #69

Closed
shruti1421 opened this issue Jan 29, 2021 · 2 comments

Comments

@shruti1421
Copy link
Contributor

Is your feature request related to a problem? Please describe.

-As we know it was very difficult to write code by using class components in a react app of having such a large code base.
-Also in many sections of the code this.state and this.props is used a lot of times so, to make the code look cleaner we can destructure the props and states.

Describe the solution you'd like

-In my opinion we can use functional components and hooks in all the components of this React App.
-Destructing of props and states wherever it is needed.
This will increase the readability of the code and make the code to look cleaner.

I would like to make this changes as mentioned above in login and register page.

@shruti1421 shruti1421 changed the title Can use hooks and functional components instead of using class components Replace class components and states with functional components and hooks Jan 29, 2021
@shruti1421 shruti1421 changed the title Replace class components and states with functional components and hooks feat: Replace class components and states with functional components and hooks Jan 29, 2021
@ayan-biswas0412
Copy link
Contributor

@shruti1421 I also agree with you , I want to contribute in this issue also for the other pages in the repo.

@Aaishpra
Copy link
Member

Duplicate of #76

@Aaishpra Aaishpra marked this as a duplicate of #76 Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants