Skip to content
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

"Deny when out of stock" behavior improvements #1928

Closed
brent-hoover opened this issue Mar 2, 2017 · 3 comments
Closed

"Deny when out of stock" behavior improvements #1928

brent-hoover opened this issue Mar 2, 2017 · 3 comments
Assignees
Milestone

Comments

@brent-hoover
Copy link
Collaborator

brent-hoover commented Mar 2, 2017

I just noticed two issues with this behavior that probably need to be fixed. One minor and one sort of important.

  1. If I have 5 in stock and I add 10 of something to my cart my quantity is reduced to 5 (which makes sense) but I don't receive any sort of alert that my quantity has been changed and the "Added to Cart" alert shows the original quantity, not the adjusted quantity. This could lead to user confusion. I feel like at one time there was an alert that said "Your quantity has been adjusted".

  2. If I have 5 in stock and I add 10 to my cart my quantity is reduced to 5. If I add another 20, the total quantity is now 10. So the "Deny when out of stock" is only the affecting the quantity I can add to cart, not the quantity in my cart. So I could have 10 in my cart even though there are only 5 in-stock, which I would think is exactly what "Deny when out of stock" is trying to avoid.

@joykare
Copy link
Contributor

joykare commented Mar 17, 2017

Related to #1935

@joykare
Copy link
Contributor

joykare commented Mar 21, 2017

Added an alert that pops up when quantity is adjusted. Working toward changing the checkout pop up to signal exactly how many items have been added to cart.

@brent-hoover
Copy link
Collaborator Author

Closed via #2034

@ghost ghost removed the review label Apr 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants