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

Add plurals dropped from rails-i18n in 7.0.6 #266

Merged
merged 1 commit into from
Nov 21, 2022
Merged

Conversation

kevindew
Copy link
Member

The 7.0.6 release of rails-i18n dropped support for a number of plurals 1 and this causes our apps that make use of these languages to error and thus be unable to update to rails-i18n 7.0.6 (example: alphagov/whitehall#7042)

This commit ports over these rules from rails-i18n to maintain the behaviour.

There is also something of a duplicate of this file in rails_translation_manager that may also need an update. I'm unsure of the status of that file though as it is already out-of-sync with this one and I think this one takes priority.

The 7.0.6 release of rails-i18n dropped support for a number of plurals
[1] and this causes our apps that make use of these languages to error
and thus be unable to update to rails-i18n 7.0.6

This commit ports over these rules from rails-i18n to maintain the
behaviour.

There is also something of a duplicate of this file in
[rails_translation_manager](https://github.com/alphagov/rails_translation_manager/blob/main/config/locales/plurals.rb)
that may also need an update. I'm unsure of the status of that file
though as it is already out-of-sync with this one and I think this one
takes priority.

[1]: svenfuchs/rails-i18n#1017
chao-xian added a commit to alphagov/rails_translation_manager that referenced this pull request Nov 16, 2022
The 7.0.6 release of rails-i18n dropped support for a number of plurals
[1] and this causes our apps that make use of these languages to error
and thus be unable to update to rails-i18n 7.0.6

This commit ports over these rules from rails-i18n to maintain the
behaviour.

This update follows what was added into govuk_app_config [2]

[1]: svenfuchs/rails-i18n#1017
[2]: alphagov/govuk_app_config#266
@kevindew kevindew merged commit 4a80089 into main Nov 21, 2022
@kevindew kevindew deleted the extra-plurals branch November 21, 2022 10:51
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