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

useHooks #6

Closed
behnood-eghbali opened this issue Oct 2, 2020 · 16 comments · Fixed by #9
Closed

useHooks #6

behnood-eghbali opened this issue Oct 2, 2020 · 16 comments · Fixed by #9
Labels
enhancement New feature or request good first issue Good for newcomers Hacktoberfest React

Comments

@behnood-eghbali
Copy link
Owner

Use hooks as a global state manager for this app. Use functional components instead of class components.

@behnood-eghbali behnood-eghbali added enhancement New feature or request good first issue Good for newcomers Hacktoberfest React labels Oct 2, 2020
@faabiopontes
Copy link
Contributor

May I help with that?

@behnood-eghbali
Copy link
Owner Author

@faabiopontes Sure!

@ruhci28
Copy link
Contributor

ruhci28 commented Oct 2, 2020

Hii @behnood-eghbali can i change the contact.js and message.js to functional component.?

@behnood-eghbali
Copy link
Owner Author

@ruhci28 Sure. :) Why not? ;)

@faabiopontes
Copy link
Contributor

@ruhci28 I will not take those then

@ruhci28
Copy link
Contributor

ruhci28 commented Oct 2, 2020

yeah sure @faabiopontes

@Guilh
Copy link

Guilh commented Oct 2, 2020

I'd love to take some on as well if there's room. :)

@behnood-eghbali
Copy link
Owner Author

@Guilh Ok :) What are you suggesting?

@faabiopontes
Copy link
Contributor

@behnood-eghbali maybe do a checklist expliciting everything that can be changed (all components, all hooks) and then each can take one

@behnood-eghbali
Copy link
Owner Author

@faabiopontes Good point! :)

@behnood-eghbali
Copy link
Owner Author

@faabiopontes You can set the global state with hooks and change the structure. @Guilh You can change the components.

@faabiopontes
Copy link
Contributor

@behnood-eghbali will do

@yigitcukuren
Copy link
Contributor

I have made some changes, please see #9.

@felix-chin
Copy link

I would love to help with this as well, if there's still work to go around

@behnood-eghbali
Copy link
Owner Author

@yigitcukuren Thank you! :)

@behnood-eghbali
Copy link
Owner Author

@felix-chin I'm so sorry but I have to close this issue. :( If there's any changes or issues I'll let you know. Thank you! ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers Hacktoberfest React
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants