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

Virtual products and variants #600

Closed
panakour opened this issue Oct 10, 2020 · 11 comments
Closed

Virtual products and variants #600

panakour opened this issue Oct 10, 2020 · 11 comments

Comments

@panakour
Copy link
Contributor

Is there any reason in which virtual products not support variants?

Lets say a ticket product (an event) that should has various variations depending on the date of the event.

@tobias-kuendig
Copy link
Member

If I remember correctly, it was done to keep the implementation small (I guess some virtual product support is better than none?). There is no other technical reason and I feel like it would make sense to add variants to virtual products as well.

@tobias-kuendig
Copy link
Member

I'm closing this issue due to inactivity. If you need any further assistance feel free to reply.

If someone wants to implement this feature Pull Requests are always welcome.

@chocolata
Copy link
Contributor

Hi,

I just came across this. In my case the shop is selling memberships. There is one membership with three variants, because there are three types of memberships. The product description remains the same for the three memberships, so I guess it would be logical to do it that way.

I'm not using the virtual file though. Would it suffice to disable shipping and not use the Is virtual flag then and instead just uncheck the "shippable" flag?

@chocolata
Copy link
Contributor

Just investigated further and it seems that disabling the "shippable" flag still applies shipping costs to the order. That shouldn't happen, right?

So both in the cart and in the quickcheckout I see the default shipping method price added to the total. And the shipping method selector is present.

Variants or no variants: the issue seems to persist.

When I go over the code, I see that only the is_virtual flag is used not to unset the shipping method. Is this something we can change?

@chocolata
Copy link
Contributor

chocolata commented Feb 19, 2024

Hi, I created a pull request to allow variants even when is_virtual is set to true.

I personally am not using the file download function, so that should be thoroughly tested. Could you kindly consider this?

As an alternative, we might reconsider the is_shippable flag, and have products without this flag not require a shipping method.

What are your thoughts?

@chocolata
Copy link
Contributor

Hi @tobias-kuendig , did you have a chance to review this?

@SamBrishes
Copy link
Collaborator

Hello @chocolata,
I've some time left today, I'll take over the review.

@SamBrishes
Copy link
Collaborator

Pushed to develop

I have made a few more adjustments, mainly targeting the product files when using different variants. However, could successfully test variants on digital products with and without product files attached.

PS.. Product Files can now also be limited to specific variants

image

@chocolata
Copy link
Contributor

Looks fantastic! Thanks Sam!

Could we also review the functionality of the "shippable" flag? My clients are finding it unclear when to select this option, particularly for events or memberships. Despite unchecking the "shippable" option, the checkout process still includes shipping costs. It appears that only the "is_virtual" flag exempts an item from shipping charges, which seems counterintuitive. Could we have a look at this behavior and see if it would be beneficial to change it?

@SamBrishes
Copy link
Collaborator

Definitely, I'll have a look at it soon. (Moved to new issue #1115 )

@chocolata
Copy link
Contributor

Much appreciated... Thanks a bunch!

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

No branches or pull requests

4 participants