Skip to content

Issues: Morton358/weatherApp

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

11 Open 18 Closed
11 Open 18 Closed
Author
Filter by author
Loading
Label
Filter by label
Loading
Use alt + click/return to exclude labels
or ⇧ + click/return for logical OR
Projects
Filter by project
Loading
Milestones
Filter by milestone
Loading
Assignee
Filter by who’s assigned
Sort
Sort by
Newest Oldest Most commented Least commented Recently updated Least recently updated Best match
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀

Issues list

Your code is very hard to read, because it is too big. It looks like copy and paste
#29 opened Apr 13, 2019 by Madejczyk
1
Call setState as function not as object. Please check documentation
#28 opened Apr 13, 2019 by Madejczyk
1
Check how to call automatic dispatch method
#23 opened Apr 13, 2019 by Madejczyk
1
You install a lot dependencies which are unnecessery. Check size of application what % are dependencies and your app
#17 opened Apr 13, 2019 by Madejczyk
1
Maybe better idea is using Lerna?
#15 opened Apr 13, 2019 by Madejczyk
IF you want to this to production please not use create-react-app
#14 opened Apr 13, 2019 by Madejczyk
1
Duplication code - line 64 - sub... and unsub...
#11 opened Apr 13, 2019 by Madejczyk
1
can we create const ?
#9 opened Apr 13, 2019 by Madejczyk
1
Can we hide this information?
#8 opened Apr 13, 2019 by Madejczyk
1
Can we use fetch?
#7 opened Apr 13, 2019 by Madejczyk
1
Can we remove http npm?
#6 opened Apr 13, 2019 by Madejczyk
1
ProTip! no:milestone will show everything without a milestone.