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 #2182

Open
wants to merge 45 commits into
base: merchello-dev
Choose a base branch
from
Open

Conversation

cwjackso
Copy link

@cwjackso cwjackso commented Oct 4, 2018

Not sure if you want to merge this but it is our working "virtual products" development.

It allows product variants to be generated on the fly rather than database stored as currently - its backward compatible as there is a tick box per product to enable.

Its main use is as we discussed a long time ago around the performance of the admin area once the product option count is high.

We added some helpers in to allow for products to be created on the fly for adding to basket etc. Let me know if you want to go through anything or if merging is not something you wish to do.

Carl Jackson and others added 30 commits November 21, 2017 12:07
fixed migration for virtual products column
fixed saving virtual variants flag on a product
@YodasMyDad
Copy link
Contributor

Thanks for doing this. I'll leave this open for now. As I think it's an edge case.

@cwjackso
Copy link
Author

Understand however the PR does not change default functionality. It just adds an alternate/lazy method of generating variants so is worth considering.

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