-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
Localization for Brazilian Portuguese (pt-br) #1785
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1785 +/- ##
================================================
- Coverage 62.6278% 61.6161% -1.0117%
================================================
Files 168 169 +1
Lines 25032 25443 +411
Branches 1446 1447 +1
================================================
Hits 15677 15677
- Misses 9302 9712 +410
- Partials 53 54 +1 ☔ View full report in Codecov by Sentry. |
Thanks! Just fix the formatting with |
Sure, I will make the changes and commit. In my future contribuitions I'll remember to do that before PR. Thank you! |
Yarn format is not a script in package-json, so I used |
Yup, thats what i meant. |
Passing run #1990 ↗︎Details:
Review all test suite changes for PR #1785 ↗︎ |
This pull request introduces a new locale file and option in locale menu for Brazilian Portuguese (pt-br). I took the liberty of not only translating but also localizing some expressions to make it more appealing to the Brazilian audience.
Files Modified:
(new) client/src/locales/pt-br.ts
(updated) client/src/components/navbar/LocaleSelector.vue
Tested with some friends.