Skip to content

Loading…

i18n Vendored Rails Translations #173

Closed
dirkkelly opened this Issue · 7 comments

4 participants

@dirkkelly

Looks like the vendored rails is still using the double moustache syntax of translations, eg:

rails/actionpack/lib/action_view/locale/en.yml

This is fixed in 2.3.10 of actionpack, could we look at upgrading edge?

@saturnflyer
Radiant CMS dev team member

As far as I understand, there is a bug in a version of rack that rails requires in 2.3.10 which truncates the contents of textareas.

@dirkkelly

Ahhh lame. I was looking for a reason why we hadn't upgraded.

Should I manually fix these and send a pull request...at least that fixes the immediate issue?

@saturnflyer
Radiant CMS dev team member

that'd be fine with me.

@spanner

I tried to put in a sort of monkeypatch for the i18n and found it's not that easy. The much easier option is to downgrade your i18n gem to 0.3.7, if other projects will allow. Perhaps we should vendor that, temporarily?

@dirkkelly

cheers, i'll see what I can sort out :-)

@michaelstalker

You can create a preinitializer.rb file and stick it in your config folder to take care of this. See http://www.taylorbrooks.org/error-messages-in-rails-attribute-message/

@spanner

We're on 2.3.11 now so this shouldn't be a problem any more.

@spanner spanner closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.