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

localization of the site to Brazilian Portuguese #68

Merged
merged 6 commits into from
Mar 21, 2021

Conversation

augusto-herrmann
Copy link
Contributor

@augusto-herrmann augusto-herrmann commented Mar 20, 2021

A draft PR so we can start translating the site to Brazilian Portuguese. When all is done and reviewed, we shall submit the pull request.

@augusto-herrmann
Copy link
Contributor Author

I think I'm now done adding everything needed for the translation.

@gaugsouza, feel free to review it before we send the PR.

@augusto-herrmann augusto-herrmann marked this pull request as ready for review March 21, 2021 04:04
@JustinBack
Copy link
Member

Awesome! I'll merge it so it will soon appear on the production environment!

@JustinBack JustinBack merged commit cb22a65 into tosdr:master Mar 21, 2021
@augusto-herrmann augusto-herrmann deleted the l10n_pt_BR branch March 22, 2021 12:55
@augusto-herrmann
Copy link
Contributor Author

Thanks, @JustinBack !

It appears the deploy has already taken effect, considering that Brazilian Portuguese now appears as an option. But for some reason the translated strings are not loading and we get English instead.

front-page-pt-br-translation

What could be wrong?

I took the en.json file as a model. Copied it and translated it.

@JustinBack
Copy link
Member

I believe the issues lies in postgres only storing data in lowercase where as pt_BR contains uppercase letters and not finding it (can be reproduced on mexican spanish), I'll deploy a fix

@JustinBack
Copy link
Member

The issue has been fixed, thanks for reporting!

@JustinBack JustinBack added the translations Improvements or additions to translations label Mar 22, 2021
@augusto-herrmann
Copy link
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translations Improvements or additions to translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants