Translation of strings.xml in Portuguese (BR)

[thezakman]

Describe the pull request

Portuguese (Brazilian) translation of strings.

Link to relevant issues

Issue #430 Translations needed

[thezakman]

#430 (comment)

[corbinlc]

@thezakman Please notice this error in the build log:

> Task :app:mergeDebugResources FAILED
/home/circleci/workspace/app/src/main/res/values-pt-BR: Error: Invalid resource directory name

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:mergeDebugResources'.
> /home/circleci/workspace/app/src/main/res/values-pt-BR: Error: Invalid resource directory name

[corbinlc]

Probably want, values-pt-rBR instead of values-pt-BR. Please try that out.

[thezakman]

Ok, ill push it now.

[thezakman]

@corbinlc still got error with "values-pt-rBR"

[corbinlc]

The error is different this time. You now have the filed named correctly.

> Task :app:mergeDebugResources FAILED
/home/circleci/workspace/app/src/main/res/values-pt-rBR/strings.xml:61:39: Error: Element type "string" must be followed by either attribute specifications, ">" or "/>".

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:mergeDebugResources'.
> /home/circleci/workspace/app/src/main/res/values-pt-rBR/strings.xml:61:39: Error: Element type "string" must be followed by either attribute specifications, ">" or "/>".

[corbinlc]

You have a syntax issue on line 61

[corbinlc]

<string name="illegal_state_title"UserLAnd entrou em um estado ilegal!</string>
should probably be
<string name="illegal_state_title">UserLAnd entrou em um estado ilegal!</string>

[corbinlc]

Please click on a failing build where it says "Details" to see the same errors I am seeing.

[thezakman]

@corbinlc my bad, that is true.
Will fix it now.

[thezakman]

Done ;)

[corbinlc]

Do you think this translation would work pretty well for Portuguese speakers in other places outside of Brazil?

[thezakman]

@ corbinlc, will work 90% since European Portuguese (Portugal) differs somewhat from ours (not much though).

[corbinlc]

I was just thinking we could make this the values-pt file instead of the values-pt-rBR file and help people outside of Brazil too. Then, we would only create a values-pt-r* for differences that people flag. What do you think?

[thezakman]

It is better to keep European Portuguese as a standard and use values-pt-rBR ​​for Brazilians, since it is used not only in Portugal but also in other countries in Africa.

[corbinlc]

We just don't have any European Portuguese translation so far.

[thezakman]

So if you think would be better to keep this format, go for it :)

[corbinlc]

All looks good now. Obrigado.

[corbinlc]

This will go out in our next release.

[thezakman]

No problem man, i'm glad I could help.
Cheers!

[hugok79]

We just don't have any European Portuguese translation so far.

@corbinlc I can help with the European Portuguese translation. (pt, pt_PT)