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

feat(i18n): remove i18n fallback config in prod #451

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

difernandez
Copy link
Contributor

@difernandez difernandez commented Jul 31, 2023

Contexto

Potassium agrega i18n y es-CL o en como idioma por defecto. También agrega un par de fallbacks.

Este PR

Este PR saca una configuración que se agregaba por defecto en production.rb, config.i18n.fallbacks = true. Potassium ya configura los fallbacks en el application.rb con otro valor, entonces estaba pasando que la configuración en producción y local eran diferentes.
En la mayoría de los casos no es mucho problema, ya que en ambos toma el idioma seleccionado, es-CL. Pero en caso de no tener una traducción, en local pasaba a los fallbacks es y en, mientras que en producción tiraba un error de translation missing

Use application.rb config, same as in development
@difernandez difernandez requested a review from fprebolledo July 31, 2023 18:56
Copy link
Contributor

@fprebolledo fprebolledo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💎

@difernandez difernandez merged commit 74a896b into master Aug 1, 2023
@difernandez difernandez deleted the use-same-fallback-in-every-env branch August 1, 2023 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants