-
Notifications
You must be signed in to change notification settings - Fork 16
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
Mw cart style #73
base: master
Are you sure you want to change the base?
Mw cart style #73
Conversation
Add retire product method to Products Controller, updated in prod index view
OAuth authentication functionality
Jrmerchantdashboard2
Nl products new create
Nl products new create
…orderitem model, and worked on styling home page and nav bar
…n in the merchant dashbosard
Mw Deleted filter by pending order status from merchant dashboard
Added Merchant model test, updated merchant dashboard product views
added review seed
update gitignore file to include coverage folder and seed file
Mw Filtering out pending orders from merchant order fulfillment page in dashboard
bEtsyWhat We're Looking ForManual testing
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 y'all:
I do see some room for improvement around:
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. |
No description provided.