-
Notifications
You must be signed in to change notification settings - Fork 13
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
Category views #2
base: master
Are you sure you want to change the base?
Conversation
category options
bEtsyFunctional Requirements: Manual Testing
Major Learning Goals/Code Review
Overall FeedbackGreat work overall! You've built a fully functional web store from top to bottom. This represents a huge amount of work, and you should be proud of yourselves!. I am particularly impressed by the way that you thoughtfully constructed both the front end by making navigation straightforward, and put together code on the back end that was easy to read and cross-reference! The logical consistency in the back end made code navigation a breeze for me. I do see some room for improvement around asking clarifying questions on requirements! There are a couple of points that were deducted because the app didn't behave as expected or because I couldn't find what I was looking for (especially frustrating because of how easy the rest of navigation was!). If there is any question about where functionality should live, or what page buttons should be on, that's an opportunity to write it down and bring it to your PM! bEtsy is a huge project on a very short timeline, and this feedback should not at all diminish the magnitude of what you've accomplished. Keep up the hard work! Only the person who submitted the PR will get an email about this feedback. Please let the rest of your team know about it. |
Assignment Submission: bEtsy
Congratulations! You're submitting your assignment. Please reflect on the assignment with these questions. These should be answered by all members of your team, not by a single teammate.
Reflection