Swagrid/master #15
Swagrid/master #15
Conversation
Swagrid/cart merge
Swagrid/emily
…ting in commented out code for orders controller spec
Swagrid/login
| end | ||
| end | ||
|
|
||
| # This method will let an admin cancel an order, but we don't have |
There was a problem hiding this comment.
It's good that you guys want to show that you attempted something, but generally shouldn't leave this in. However, if your instructors have been cool with you guys doing it like this then don't mind me.
|
Visually, your site was very pleasing and navigationally intuitive. Love the attention to detail for difference between form incrementation between price and quantity. I did find that there was a small "updating" bug for your order confirmation after entering your shipping address. If you got bored, a future iteration could address this and validate the address. Other than that, awesome job! |
| @@ -0,0 +1 @@ | |||
| 2.2.3 | |||
There was a problem hiding this comment.
Okay, this might be seem small, but this is such good practice! I once had Heroku blow up on me because I was using a built-in method that was in a newer version of Ruby. 👍
…ust be logged in to retire products
No description provided.