Skip to content

Fixes plural issue with locales#3011

Merged
PeerRich merged 3 commits intomainfrom
fix/translation_plurals
Jun 8, 2022
Merged

Fixes plural issue with locales#3011
PeerRich merged 3 commits intomainfrom
fix/translation_plurals

Conversation

@leog
Copy link
Contributor

@leog leog commented Jun 7, 2022

What does this PR do?

Plurals for texts in the app were not working, due to a upgrade in the next-i18next dependency with a breaking change regarding that (more info here: https://www.i18next.com/misc/migration-guide#json-format-v4-pluralization).

Now the singular i18n key should be suffixed with _one and the plural _other.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@vercel
Copy link

vercel bot commented Jun 7, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Jun 8, 2022 at 7:18AM (UTC)
3 Ignored Deployments
Name Status Preview Updated
docs ⬜️ Ignored (Inspect) Jun 8, 2022 at 7:18AM (UTC)
swagger ⬜️ Ignored (Inspect) Jun 8, 2022 at 7:18AM (UTC)
ui ⬜️ Ignored (Inspect) Jun 8, 2022 at 7:18AM (UTC)

@zomars zomars enabled auto-merge (squash) June 7, 2022 22:58
agustif
agustif approved these changes Jun 8, 2022
@CarinaWolli CarinaWolli added the ♻️ autoupdate tells kodiak to keep this branch up-to-date label Jun 8, 2022
@PeerRich PeerRich disabled auto-merge June 8, 2022 07:32
@PeerRich PeerRich merged commit 33a4bb7 into main Jun 8, 2022
@PeerRich PeerRich deleted the fix/translation_plurals branch June 8, 2022 07:32
@zomars zomars mentioned this pull request Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ autoupdate tells kodiak to keep this branch up-to-date

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants