You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hello, I'm implementing this extension in a Spree 3-0-stable localized project.
I noticed that the product search was not returning any products (maked correctly), even though the query and scope are working correctly (search_can_be_part scope).
This is due to the extension spree_i18n version 3-0-stable because all products have a translation table (Spree::Product::Translation) and the query to search is not working the name field, so I made the following mod:
I've also had this issue and fixed in a similar manner in a specific project.
This issue originates from I18n and is a common theme across multiple gems due to the separate translations tables. A lot of Spree functionality also seems to not work properly because of this. See this discussion on Ransack queries not working with I18n.
These issues are more of concern for spree_i18n gem which should provide some kind of infrastructure for clients to be able to query the model or translation without having to know if the app is localized or not.
There were some attempts to enable Ransack for this here but as far as I can see this is still an issue.
If I18n fixes its ransack queries then this might be rewritten with ransack but that would require adding it as a dependency.
Hello, I'm implementing this extension in a Spree
3-0-stable
localized project.I noticed that the product search was not returning any products (maked correctly), even though the query and scope are working correctly (
search_can_be_part
scope).This is due to the extension
spree_i18n
version3-0-stable
because all products have a translation table (Spree::Product::Translation
) and the query to search is not working the name field, so I made the following mod:I didn't made a PR because I'm not sure if this is the correct way to code it (works for me tho) or if it's even useful for other stores.
The text was updated successfully, but these errors were encountered: