-
Notifications
You must be signed in to change notification settings - Fork 341
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
Attribute option translation #3117
Comments
Hello @Nuranto, I was able to reproduce your issue. Indeed the attribute options translation does not work through any language (e.g. We're not a huge fan of adding an emulate store somewhere in the index rebuilding process. As an easy solution, Magento proposes a way to add translation via BR, |
Hi @vahonc, It reminds me a similar question from Romain here : #2932 Here are some examples :
Maybe store emulation is not necessary. Maybe locale emulation is enough ( |
In any case, this is exactly what does these 2 modules : https://github.com/Smile-SA/magento2-module-custom-entity |
I wasn't aware about those modules, and it looks great. I'll definitely look into it, however that won't solves the other 2 cases |
… category/executeFull)
Preconditions
Magento Version : 2.4.6-p3
ElasticSuite Version : 2.11.4.1
Steps to reproduce
[ ['value'=>1, 'label' => __('Apple')->render()], ['value'=>2, 'label' => __('Car')->render()]. ]
Expected result
Actual result
Additional informations
Quick fix I found is to manually load translations in my source model :
But that's obviously not a good solution, that should be done somewhere in smile module. I did not found the correct place to do it though, we should probably emulate store somewhere in index rebuilding process.
EDIT : This solution does not work, it loads the translations, yes, but does not reload it for each store. So it just solve the problem to one store, and move the issue to the other one. So we definitely need store emulation here.
The text was updated successfully, but these errors were encountered: