Skip to content

Conversation

@pcreux
Copy link

@pcreux pcreux commented Oct 3, 2025

Fixes #1154.

ko.yml is the only translation file with "%{attribute}" in the errors.messages translations. That raises errors since rails does not pass %{attribute}.

I tasked ChatGPT/codex with fixing these translations, as I don't read Korean.

Let me know if I should update the CHANGELOG.md.

Look at #1154 for more details.

Copy link
Contributor

@sunny sunny left a comment

Choose a reason for hiding this comment

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

I don’t read Korean but technically this fixes the issue. Thank you 👍🏻

@sunny
Copy link
Contributor

sunny commented Oct 3, 2025

An entry to the CHANGELOG would be nice 🙏🏻

@pcreux
Copy link
Author

pcreux commented Oct 6, 2025

@sunny Done! Who is responsible for releasing the gem?

I need this change for Rails 7.2, and as far as I can tell, there's no rails-7 branch. So, I'll keep a fork until we upgrade to Rails 8, unless you tell me otherwise.

@sunny
Copy link
Contributor

sunny commented Oct 6, 2025

There’s no maintained Rails 7 branch that I know of.

@pama 🙏🏻

@pcreux
Copy link
Author

pcreux commented Oct 6, 2025

Here is a rails-7 branch if anyone is interested. :)

https://github.com/pcreux/rails-i18n/tree/rails-7

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.

ActiveModel#validate! fails for ko locale because errors.* keys require ${attribute}

2 participants