Skip to content
This repository has been archived by the owner on Mar 8, 2021. It is now read-only.

use hooks #10

Open
schirrel opened this issue Aug 23, 2020 · 4 comments · May be fixed by #11
Open

use hooks #10

schirrel opened this issue Aug 23, 2020 · 4 comments · May be fixed by #11
Labels
enhancement New feature or request good first issue Good for newcomers hacktober hacktoberfest help wanted Extra attention is needed

Comments

@schirrel
Copy link
Collaborator

The library was started with me, who dont know a lot of react, and since its creation was not mainteined.

I've knwo React has some new features to components, something like hooks or useSomeThing, would be nice someone to update de library to the most recent version.

Thanks

@schirrel schirrel added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed hacktoberfest hacktober labels Aug 23, 2020
@SaurabhS78
Copy link

The new hooks could be implemented in the existing components by changing them from class components to functional components. However, you might loose some functionality that class components offer because hooks is a new feature in react and might raise some compatibility issues.
Do you wish to switch to functional components?

Thanks

@schirrel
Copy link
Collaborator Author

schirrel commented Sep 1, 2020

Hi @SaurabhS78 the components are pretty outdated, so yes cand be switch ;P

@akshatshah21
Copy link

Hey @schirrel , just wanted to clarify this: you're looking to convert all class-based components to functional components with hooks right? If so, can I contribute to this?

@schirrel
Copy link
Collaborator Author

schirrel commented Oct 3, 2020

@akshatshah21 yes, go on!!

@akshatshah21 akshatshah21 linked a pull request Oct 6, 2020 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers hacktober hacktoberfest help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants