Ari Birnbaum

Results 33 comments of Ari Birnbaum

Any updates on when this might be merged? I'm still experiencing #3956.

This would be really cool to see!

It is occurring with Shopify. I'm unfamiliar with how other providers handle price for variants. I haven't tested this with any of the demos. The following change to the `price`...

Would it make more sense to do this? ```tsx variant ? variant.price : product.price.value ``` So, variant price is only applied if a variant is being shown. As I understand...

@geniti, I'm a bit confused by your comment. The snippet I commented is the solution I've made which seems to work for Shopify. Specifically, `amount: (variant ? variant.price : product.variants[0].price)`....

@geniti the version of commerce starter kit I'm using has this before the `return`: ```tsx const { price } = usePrice({ amount: product.price.value, baseAmount: product.price.retailPrice, currencyCode: product.price.currencyCode!, }) ``` Mine...

@gnacoding nice catch! I completely forgot I updated the `ProductVariant` type. Also, to answer your question, my version of vercel/commerce is two months old. So, either, the ordering of price...

Same issue here. This should be high priority.

+1 I know it's likely a design decision, but this looks like a bug.

I was able to fix this by building from source. I hope there will be a fix for the copr version at some point.