-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Updated 0 board to 0 boards (singular vs plural) #1210
Conversation
/cla |
I was thinking about something more in the line of this: https://codesandbox.io/s/o56w8ywwoz Also you shouldn't concat numbers with strings, not all the languages in the world puts the numbers first, some languages could express that like "Boards 1" for example. so try to include the whole phrase in the translation, including all the parts of the phrase, and avoid concatenations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still can be improved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still some improvements to be made.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still some things need to be handled.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Updated 0 board to 0 boards (singular vs plural * Made text translatable * Used better i18n for singular-plural * Updated tests * Minor fixes * done * Lint fix * Updated tests
Summary
Updated 0 board to 0 boards (singular vs plural)
Ticket Link
Fixes #1200