-
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
Branches - Amal, Angele, Dora, Julia #97
base: master
Are you sure you want to change the base?
Conversation
…d updated methods to be more easily testable
Add flash messages to application
Browse products by category
…ed with migration to move retired from order to product
Added hyperlinks for homepage and link style
Updated retired to correct model with migrations
update product stock validation to be greater than zero
fixed css for forms and carousel
Overall style
Retire (merged with overall style, that one should be first)
More categories products validations
Background
Team 3 bEtsy WIP bEtsyWhat We're Looking ForManual testing
Code Review
Overall FeedbackGreat work overall! You've built a 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 y'all injected lots of your own fun creativity into this project and coordinated so well together on a whole team with 4 members pushing to the same repo, especially given it's y'all's first time! Also y'all's route organization is great! I do see some room for improvement around more thorough edge case testing (both manual testing and unit testing) and putting a bit more logic into the model. (Though I will say that in general it seems like y'all made a good habit of that.) 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. |
bEtsy
Congratulations! You're submitting your assignment! These comprehension questions should be answered by all members of your team, not by a single teammate.
Comprehension Questions
determine_wizard
controller filter and how it's used for authorization, or drying up the form to use it for both new and edit (wasn't able to do) ; Dora: OAuth login and why it didn't work as well once on heroku (Jared knows the issue); Julia: wizard dashboard