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

Make it possible to render DSL levels without translations #27953

Merged
merged 3 commits into from Apr 10, 2019
Merged

Conversation

Hamms
Copy link
Contributor

@Hamms Hamms commented Apr 9, 2019

Right now, all DSL levels depend on I18n.t to render their content, and
will fall back to nil if they can't find the translation. We want to
be able to only sync data into the translation system if we want to
actually translate it, so we add the ability for DSL levels to fall back
to the database content if they can't find translation data.

Originally added as part of #27881, which also includes changes to the way the DSL i18n data is synced. Extracted into this PR for a clearer division of concerns

Right now, all DSL levels depend on I18n.t to render their content, and
will fall back to `nil` if they can't find the translation. We want to
be able to only sync data into the translation system if we want to
actually translate it, so we add the ability for DSL levels to fall back
to the database content if they can't find translation data.
@Hamms Hamms requested a review from joshlory April 9, 2019 20:59
Copy link
Contributor

@joshlory joshlory left a comment

Choose a reason for hiding this comment

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

Rad!

@Hamms Hamms merged commit 7a12ea3 into staging Apr 10, 2019
@Hamms Hamms deleted the dsls-fallback branch April 10, 2019 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants