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

Catalog variant isSoldOut and isLowQuantity are always false #4741

Closed
mikemurray opened this issue Oct 15, 2018 · 1 comment · Fixed by #4896
Closed

Catalog variant isSoldOut and isLowQuantity are always false #4741

mikemurray opened this issue Oct 15, 2018 · 1 comment · Fixed by #4896
Labels
bug For issues that describe a defect or regression in the released software

Comments

@mikemurray
Copy link
Member

Issue Description

The denormalized inventory flags for isSoldOut and isLowInventory are always false for variants in the catalog.

Steps to Reproduce

  1. Go to a product
  2. Set variant to the quantity to 0
  3. Turn off backorders
  4. Publish
  5. Look in mongo for that Catalog Variant
  6. See that isSoldOut is false even though it should be true

Possible Solution

In createCatalogProduct.js function xformVariant(variant, variantPriceInfo, shopCurrencyCode, variantMedia) always receives undefined for variants.

xformVariant will have to do some additional processing to determine the correct inventory flags and not assume they come from the params.

Versions

Node: 10.7.0
NPM: 6.2.0
Meteor Node: 8.11.4
Meteor NPM: 6.4.1
Reaction CLI: 0.29.0
Reaction: 2.0.0-rc.5
Reaction branch: fix-4722-mikemurray-compare-at-price
Docker: 18.06.1-ce

@mikemurray mikemurray added the bug For issues that describe a defect or regression in the released software label Oct 15, 2018
@mikemurray mikemurray added this to the Pikes Peak milestone Oct 15, 2018
@spencern
Copy link
Contributor

spencern commented Dec 8, 2018

@mikemurray can we close this? I see we merged #4742.

@spencern spencern removed this from the Pikes Peak milestone Dec 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug For issues that describe a defect or regression in the released software
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants