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

Cart items picture displays placeholder (sometimes) when product has variants #3263

Closed
speedingdeer opened this issue Nov 9, 2017 · 5 comments
Assignees
Labels
bug For issues that describe a defect or regression in the released software

Comments

@speedingdeer
Copy link

speedingdeer commented Nov 9, 2017

Expected behavior

Cart displays the selected product variant's picture

Actual behavior

Cart display nothing (sometimes) instead of selected product variant's picture

Steps to reproduce the behavior

Create few product variants and add pictures for each of them. Next as user select few and them to cart. Click on the cart image and check if the pictures in cart match the ordered variant pictures.

screen shot 2017-11-09 at 12 53 34

Versions

(run reaction -v from your reaction directory)
Node: 8.1.4
NPM: 5.0.3
Meteor Node: 4.8.5
Meteor NPM: 4.6.1
Reaction CLI: 0.20.0
Reaction: 1.5.6
Reaction branch: master
Docker: 17.09.0-ce

@brent-hoover
Copy link
Collaborator

I'm seeing this as well. It has to do with ReactionProduct.selectedVariant() is sometimes returning null. Trying to get this solved today.

@brent-hoover
Copy link
Collaborator

Which is just caused by the fact that sometimes no variant is selected, and sometimes the top-level variant is selected. If you select a variant, then the image is shown. And refreshing seems to always select the top-level variant.

@brent-hoover
Copy link
Collaborator

I believe this was fixed via #3267 but needs to be verified

@brent-hoover
Copy link
Collaborator

@codejockie Did you verify that this issue still occurs on master?

@codejockie
Copy link
Contributor

@zenweasel yes I did that. It still occurs on master.

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

No branches or pull requests

3 participants