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

Master product demo #113

Closed
mihaisolomon opened this issue Mar 27, 2023 · 7 comments
Closed

Master product demo #113

mihaisolomon opened this issue Mar 27, 2023 · 7 comments
Assignees

Comments

@mihaisolomon
Copy link

Hey,

I installed the demo application I added a master product with variants but could not see it in the frontend. Any idea how I can see it? Or it's not available in the demo?

@fulopattila122
Copy link
Member

Salut Mihai,

The issue is not on your end, currently the ProductFinder doesn't retrieve master products.
The fix is coming this week.

Are you using MySQL or some other DB engine?
Because for MySQL there's a relatively quick fix.

@mihaisolomon
Copy link
Author

I'm using MariaDb Mysql.

@fulopattila122
Copy link
Member

Yes, MariaDB works as well, of course :)

@fulopattila122
Copy link
Member

fulopattila122 commented Mar 27, 2023

Do you plan to use both plain products and masters/variants or only master/variants?

@mihaisolomon
Copy link
Author

I have both. I would prefer to use both if it's posible.
From what I saw it's either simple or master product, correct?

@mihaisolomon
Copy link
Author

Hey,

Do you have an answer for me?

@fulopattila122
Copy link
Member

fulopattila122 commented Apr 4, 2023

Hey Mihai,

sorry for the slow response. The good news is that it works now, regardless of the DB engine.

With the fresh Vanilo v3.7 release, the new ProductSearch class has been added that is a drop-in replacement of the former ProductFinder class. The most important difference between the two is that ProductSearch works with both simple products and masters products as well.

This demo repository has been updated accordingly. Please pull the latest version that is capable of handling this situation.
Most of the changes have happened in this commit 5cc9729

Thanks for your patience!

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

2 participants