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

Remove i18n-inflector-rails dependency #8420

Merged
merged 2 commits into from Jun 10, 2023

Commits on Jun 8, 2023

  1. Remove i18n-inflector-rails dependency

    There are no new releases anymore and the current version isn't
    compatible with ruby 3.x.
    
    As this feature wasn't really used a lot (Icelandic didn't even setup
    inflections properly), it's probably not worth fighting for it, so lets
    just drop it.
    
    Related to diaspora#8369
    SuperTux88 committed Jun 8, 2023
    Configuration menu
    Copy the full SHA
    6f80241 View commit details
    Browse the repository at this point in the history

Commits on Jun 10, 2023

  1. Don't modify default translations when merging with other language

    `deep_merge!` modifies the hash of the default translation, and it looks
    like `I18n.t` always returns the same instance, so after that, the
    default stays translated. So lets duplicate the hash first, before
    modifying it, this also helps because we also add more keys below, which
    probably also shouldn't be added to the original.
    SuperTux88 committed Jun 10, 2023
    Configuration menu
    Copy the full SHA
    5508401 View commit details
    Browse the repository at this point in the history