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

Change architecture for functional components #808

Open
pauloreis7 opened this issue Dec 19, 2021 · 5 comments
Open

Change architecture for functional components #808

pauloreis7 opened this issue Dec 19, 2021 · 5 comments

Comments

@pauloreis7
Copy link

The current project is using react components written in class form, which is already outdated and it is no longer advisable to write components this way. You can see more about it in this link.

Changing components written in class to function format is a very important update to bring scalability, best practices and simpler refactoring to the project.

So that future people in the community who want to contribute, but don't know this old format of creating react components, can also add to react-stickynode.

Surely this is an important change for the future of the project in all aspects.

@redonkulus
Copy link
Contributor

@pauloreis7 Thank you for opening the discussion. I agree that the code base should be modernized and optimized. Is this something you would want to contribute to?

@bigbigDreamer
Copy link

@pauloreis7 Thank you for opening the discussion. I agree that the code base should be modernized and optimized. Is this something you would want to contribute to?

I'd love to try this.

@bigbigDreamer
Copy link

Can I use TS + React hooks?

@redonkulus
Copy link
Contributor

Sure TS and hooks are a good foundation. Just ensure all existing tests pass as you refactor.

@bigbigDreamer
Copy link

Sure TS and hooks are a good foundation. Just ensure all existing tests pass as you refactor.

ok, probably today i'll get start.

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