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.

Sign up for GitHub

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

Jump to bottom

Rewrite App.jsx as dumb component #6

Closed
Kornil opened this issue Nov 9, 2017 · 2 comments
Closed

Rewrite App.jsx as dumb component #6

Kornil opened this issue Nov 9, 2017 · 2 comments
Labels
good first issue help wanted

Comments

@Kornil
Copy link
Owner

Kornil commented Nov 9, 2017

At the moment App.jsx main component is written as a class, since it does not use state, it could potentially be rewritten as a simple ES6 functional component.

I'm open to help anyone who wants to take this issue as first one to learn about React.
@Kornil Kornil changed the title Rewrite App.jsx to dumb component Rewrite App.jsx as dumb component Nov 15, 2017
@tomhoad
Copy link
Contributor

tomhoad commented Dec 12, 2017

#13 submitted!

@Kornil Kornil mentioned this issue Dec 12, 2017
Merged
@TENsaga
Copy link

TENsaga commented Dec 13, 2017

#14 Gave it a go. Regardless, thanks for the Git lesson Kornil.

@Kornil Kornil closed this as completed Dec 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tomhoad @Kornil @TENsaga