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

Handle products in cart that don't exist any more #650

Open
nkuehn opened this issue Aug 25, 2017 · 3 comments
Open

Handle products in cart that don't exist any more #650

nkuehn opened this issue Aug 25, 2017 · 3 comments
Labels

Comments

@nkuehn
Copy link

nkuehn commented Aug 25, 2017

Steps to reproduce:

  1. add a product to the cart
  2. add another product to the cart (maybe optional)
  3. delete one of them from the CTP project (completely)
  4. go to checkout (will still show the deleted product instead of a hint that it is not on sale any more)
  5. enter address.
  6. receive unspecified error message :-( Logs show that the product id is not found.

Steps to do right: already check cart consistency when entering checkout. if platform support is not good enough yet, at least remove the products from the cart or something.

@nkuehn nkuehn added the bug label Aug 25, 2017
@katmatt
Copy link
Contributor

katmatt commented Sep 13, 2017

As far as I understand this issue, @romannakoval is currently working on a fix for it in our backend. Then we shouldn't have to fix it in sunrise.

@romannakoval
Copy link

Hi @nkuehn and @katmatt !

Yes, I partially fixed it in CTP-269, and yesterday it was merged into the master.
After it will be released, the behavior will be next: during cart recalculation (prices and taxes recalc was modified) all items that become invalid for some reason will be "silently" dropped out. During the checkout cart recalculation is called and as it will now will return a valid cart, other error (I guess it will be specified now) - that cart were changed during update - will be returned.

The only situation that may be unhandled in scenario like this, is that if shipping info became invalid - recalculation will still return error. But I'm not sure how it is handled during checkout.

FYI, introducing this fix also for cart merge is in process under CTP-625

@nkuehn
Copy link
Author

nkuehn commented Sep 19, 2017

PS: are we tracking the matching platform requirement that a user doesn't actually want stuff "silently dropped" but rather wants to know what's going on?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants