Jan Berdajs
Jan Berdajs
@shioyama managed to check this yet?
@shioyama yeah I mentioned those specs are testing implementation details. Do you have a better suggestion to fix #572 then? This definitely did fix it. The issue that it creates...
I agree. But there is a case to be made that correct functioning is more important than caching. Not suggesting to merge as is, but as it stands I have...
@shioyama I managed to isolate the behavior into a failing mobility spec: https://github.com/mrbrdo/mobility/blob/b87ac2f14e29c40a4d1aa7de15e7acd0a65abc59/spec/integration/active_record_compatibility_spec.rb#L163 (if you move the `translates` calls above the defintion of the hooks, the spec will pass) Basically,...
Yeah it seems so :(
@shioyama so 3 years later, any chance we could see this supported any time soon? The friendly_id-mobility is kinda useless without this fallback support, and I'm sure it would benefit...
@shioyama For reference here is how I implemented it for my use case with ActiveRecord table backend. It's not generalized/automatic though, but when used it will use fallback locales to...
@damianlegawiec so where is `spree_api_v1`?
@stefnnn yes we could. Although your PR was not merged after 2 months. I am afraid the maintainers are sleeping? No point in putting in the work if it will...
@damianlegawiec ping, this is kinda important. Been using these fixes in production for a while now, no issues at all.