You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Good job on using API's to fetch the data for website ionstead of just static content
cons
After login, all the tabs show the same page.
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
UI
Pros
UI looks good , looks attractive , good job on that part .
The wireframe looks great overall.
The navigation is smooth on the site.
latency is within limits on event fires.
Functionality those are included works fine although some enhancements could've done .
Redux arch is used properly , kudos on that .
Hooks understanding is also on point .
Standard Coding practices are used .
Cons
Some links are broken(e.g. the Trends tab after login)
No comments in code / Always include comments
Backend
Good job on data validation.
Good job on authentication.
Use of API's for fetching data on website.
The text was updated successfully, but these errors were encountered:
#Functionality Wise
Pros
cons
interesting and functional than a website with n different things out of which only few works.
#Coding
UI
Pros
Cons
Backend
The text was updated successfully, but these errors were encountered: