-
Notifications
You must be signed in to change notification settings - Fork 2
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.
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
Project Review #29
Comments
Email verification checks fixed |
console.log commented or removed |
Alerts are being used in authentication flow fixed or removed |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hey Team,
First, I would like to congratulate you all on completing your project. I've taken a closer look at the features you implemented and learned a lot from this project. With my experience, I came up with certain key areas that I felt you should work on to improve the project quality, structure, and efficiencies in working on a team project like this.
Overall project review points:
Code Quality:
Code Functionality
Frontend react + Redux
Backend
GitHub commits & Documentation
Architecture & wrong patterns
The text was updated successfully, but these errors were encountered: