-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Use column_fallback plugin for storing default locale's translations in original columns #12040
Use column_fallback plugin for storing default locale's translations in original columns #12040
Conversation
* Use original column for the translatable fields in the default locale * Remove migrations for transferring and deleting translatable data * Fixes #12019
… using translations * Update API V2 Storefront Products specs
@@ -28,6 +28,10 @@ def set_fallback_locale | |||
Spree::Locales::SetFallbackLocaleForStore.new.call(store: current_store) | |||
end | |||
|
|||
def default_locale |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@damianlegawiec Shouldn't we assign the default_locale
to I18n.default_locale
here: https://github.com/spree/spree/blob/main/core/lib/spree/core/controller_helpers/locale.rb#L21?
If a store has the default_locale
set to pl
then the current_locale
will also be pl
(unless picked otherwise). So we're still not using any translations and the I18n.default_locale
should also be pl
.
d051d79
to
129700b
Compare
Scope: