-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
-
Functionality Wise
- wireframing looks good.
- No client side data validation.
- On a meta level , Always remeber if certain feature you haven't implemented properly then just remove it. A Website with few working features is more
interesting and functional than a website with n different things out of which only few works.
-
Coding
- looks good , looks attractive , good job on that part .
- The wireframe looks great overall.
- The navigation is smooth on the site.
- Redux arch is used properly , kudos on that .
- Hooks understanding is also on point .
- Standard Coding practices are used .
- Not much comments in code / Always include comments
- I was not able to login.
- No client side data validation
Metadata
Metadata
Assignees
Labels
No labels