Skip to content

Conversation

@movermeyer
Copy link
Contributor

@movermeyer movermeyer commented Aug 23, 2022

What are you trying to accomplish?

Fixes #514. See #635 for discussion of how this situation came to be.

What approach did you choose and why?

Simply a revert of the changes to Backend::Base in #503, similar to the revert done for Backend::Pluralization in 1b5e345.

The impact of these changes

Keys nested within pluralization contexts will once again work for those using Backend::Base

@movermeyer movermeyer changed the title Revert #503 changes for Backend::Base [Blocked on #635] Revert #503 changes for Backend::Base Aug 23, 2022
@radar
Copy link
Collaborator

radar commented Oct 6, 2022

@movermeyer Please rebase.

@movermeyer movermeyer force-pushed the movermeyer/nested_pluralization_with_base_backend branch from 4c5687e to cda0ea5 Compare October 7, 2022 15:14
@movermeyer movermeyer changed the title [Blocked on #635] Revert #503 changes for Backend::Base Revert #503 changes for Backend::Base Oct 7, 2022
@movermeyer movermeyer marked this pull request as ready for review October 7, 2022 15:17
@movermeyer
Copy link
Contributor Author

@movermeyer Please rebase.

@radar 👍 Done.

@radar radar merged commit 42052e8 into ruby-i18n:master Feb 3, 2023
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.

Translations returned as hash when model-inflection has nested keys

2 participants