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

Problem when children of configurable products are not visible individually #3

Open
cptX opened this issue Jun 5, 2023 · 7 comments
Labels
2.4.5 bug Something isn't working

Comments

@cptX
Copy link

cptX commented Jun 5, 2023

Hi, I just installed your module and I'm trying to figure out how to use it. The first thing I noticed is that it lists all simple products even if they are children of configurable products. This then includes a url for this products that is not valid as they are not visible individually.
How can I solve this issue?

Also the generated files are .xml not .csv and are located in pub/media/run_as_root/feed
I suggest the readme to get updated

@riconeitzel
Copy link
Member

readme updated ✓

@cptX
Copy link
Author

cptX commented Jun 5, 2023

Thanks for the quick answer! Any ideas about my other question? By the way, thanks for your module!

@riconeitzel
Copy link
Member

Thanks for the quick answer! Any ideas about my other question? By the way, thanks for your module!

looking into it these days. We'll come back with more info later. Thanks for your patience!

@cptX
Copy link
Author

cptX commented Jun 5, 2023

Thanks in advance! It's positive that you aknowledge the issue. I suggest to check the existing option of the system and if the children of configurable products are set to not be visible individually to not be included...

Hm... I'm rethinking this. Maybe we should show all the individual variants with their stock, availability and separate SKU but put the link only of their corresponding parent configurable product.

@riconeitzel riconeitzel added verification required Issue must be verified as bug 2.4.5 labels Jun 5, 2023
@riconeitzel
Copy link
Member

Note:

foreach ($childProducts as $childProduct) {
here we should add a check for product's visibility and skip all 'not visible individually' products as well.

@riconeitzel
Copy link
Member

Hm... I'm rethinking this. Maybe we should show all the individual variants with their stock, availability and separate SKU but put the link only of their corresponding parent configurable product.

Usually the process is as follows:

configurable product is visible in catalog only, child products are visible in search only. product feed websites only link to specific variants (child products) as customers usually search/find the actual variant and want to be taken to that variation in the store. If they're taken to the configurable they have to configure it again -> might drain conversion rate.

anyway: linking 'not visible individually' products is a bug and must be fixed.

@riconeitzel riconeitzel added bug Something isn't working and removed verification required Issue must be verified as bug labels Jun 5, 2023
@cptX
Copy link
Author

cptX commented Jun 16, 2023

In my opinion as this is how it is with magento configurable products I suggest we lead the feed to the configurable product. I don't think we have another option. But the image shown in the feed could be the one of the variant... It's not clear to me what is the best option here. For example what title should then the feed have? Nobody changes the variant title. It is the config product title with a variant word attached with dash to the original title. Not very nice to show it in the feed. Why don't we show only the configurable product and it's image in the feed and ignore the variants completely? It's because we will not have the availability and the price given to the variant or because we may lose conversion rate because we don't show all the versions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.4.5 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants