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

Revert #503 changes for Backend::Base #637

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