-
Notifications
You must be signed in to change notification settings - Fork 2
Open
Description
I found some discrepancies in the react app tutorial contained in this repo. I would like to point those out:
- No Support for HOC : I can directly access the url http://localhost:3000/todo without the need for registering (which is bad). We can implement HOC that will take care of this.
- No Alert facility : The signin/signup components do not provide the facility for checking completeness. Although the button does not work, looks good if we have a completeness feature enabled.
- Documentation: The
Readmeis also a one-liner. We can improve upon it.
I can do the needful given some time (probably after my mid-semester exams get over in college).
Please revert the same.
Metadata
Metadata
Assignees
Labels
No labels