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

Update the translations - add pt-BR (Brazilian Portuguese) #488

Merged
merged 5 commits into from
Feb 8, 2021

Conversation

beatrizmilz
Copy link
Contributor

Hi @gadenbuie ! I'm sending a new version of #475!

(I did not add anything in the news files... only updated the yml file for the translation..)

@gadenbuie
Copy link
Member

Thank you @beatrizmilz! For compatibility with bootbox, I'd like to change the language key to pt. Do you know if the translation would differ for pt-PT (Portuguese as spoken in Portugal)?

@beatrizmilz
Copy link
Contributor Author

Thank you @gadenbuie ! About the differences between Portuguese from Portugal and Brazil, I'm not sure. I think it would be good if someone could review it.

Hi @DanielAlvesLABDH , do you know anyone that would be interested to review the language in this PR? Thank you!

@DanielAlvesLABDH
Copy link

Hi @beatrizmilz , I think the Portuguese used is correct. Didn't found any mistakes and the differences for pt-PT are minor and not relevant

@gadenbuie
Copy link
Member

Thank you @beatrizmilz and @DanielAlvesLABDH. I'm excited that learnr can now speak Portuguese!

@gadenbuie gadenbuie merged commit 859dab2 into rstudio:master Feb 8, 2021
@gadenbuie
Copy link
Member

@beatrizmilz @DanielAlvesLABDH Eu deveria ter dito: Estou animado que agora learnr pode falar Português!

@beatrizmilz
Copy link
Contributor Author

Thank you both @gadenbuie and @DanielAlvesLABDH !

Também estou animada que agora podemos criar tutoriais em Português :D

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

Successfully merging this pull request may close these issues.

None yet

3 participants