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

added I18n in views and changed some formatting #2820

Closed
wants to merge 4 commits into from
Closed

added I18n in views and changed some formatting #2820

wants to merge 4 commits into from

Conversation

PsychoPhobic
Copy link

No description provided.

@lucasmazza
Copy link
Contributor

Thank you for your interest in contributing to Devise but we have decided to not have internationalized views within Devise since they make it harder for people that do not need an internationalized application (which are the majority of the cases) to maintain their views.

Duplicated of #1642 #1983 #1989 #1699 #2058 and #2571.

@lucasmazza lucasmazza closed this Jan 3, 2014
@PsychoPhobic
Copy link
Author

Hiho,

why it is I18n harder (for people that do not need an internationalized application) when this uses default values?
so if the user dont create translation files it would also work with standard english values like atm.

@josevalim
Copy link
Contributor

@PsychoPhobic because their views is now full of t(".something") instead of the actual text.

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.

None yet

3 participants