Skip to content

Update pt_BR translation for 3.7.0 (see jamulussoftware#1086)#1141

Merged
ann0see merged 2 commits into
jamulussoftware:masterfrom
melcon:master
Mar 1, 2021
Merged

Update pt_BR translation for 3.7.0 (see jamulussoftware#1086)#1141
ann0see merged 2 commits into
jamulussoftware:masterfrom
melcon:master

Conversation

@melcon

@melcon melcon commented Feb 27, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@hoffie hoffie mentioned this pull request Feb 27, 2021
39 tasks

@hoffie hoffie left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for your work!

  • Listed in the Contributors UI window
    "<p>Melcon Moraes (<a href=\"https://github.com/melcon\">melcon</a>)</p>"
  • Changes look consistent
  • No untranslated strings (except the copyright, which is fine I'd say)

Comment thread src/res/translation/translation_pt_BR.ts Outdated
@hoffie hoffie added this to the Release 3.7.0 milestone Feb 27, 2021
@hoffie hoffie requested a review from ann0see February 27, 2021 22:07

@ann0see ann0see left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look good to me too. I'd remove the comments concerning the https:// question since I doubt the tool tip is clickable

Removing translator comments regarding https:// prefix on tooltip
@ann0see ann0see self-requested a review March 1, 2021 11:17
@ann0see ann0see merged commit 8614347 into jamulussoftware:master Mar 1, 2021
@pljones pljones moved this from Triage to Done in Tracking Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants