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

[pt-rBR] Add Portuguese Brazilian #25

Merged
merged 5 commits into from
Jul 17, 2024
Merged

Conversation

igormiguell
Copy link
Contributor

@igormiguell igormiguell commented Jul 8, 2024

I modified some small strings in English and Russian. If it is not correct tell me to correct it.

Imagery translation

Original text Translation
Quick Settings Tile Bloco de configurações rápidas
Home Screen Widgets Widgets na tela inicial
Widget Customization Personalização de widgets
Instant Connection Conexão instantânea
Root or Shizuku Root ou Shizuku
OneUI Integration Integração da OneUI

Rules and agreements

  • My commit messages comply with Conventional commits (use your IDE's Git manager to edit the commit messages if they don't)
  • Original sources of code and imagery (if any) are properly credited
  • I agree to keep the discussion under this pull request polite and on point
  • I have read translate.md and this pull request complies with it
  • This pull request does not translate license.md and translate.md
  • My translations are of good quality and does not contain inappropriate content

Copy link
Owner

@Smooth-E Smooth-E left a comment

Choose a reason for hiding this comment

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

It is nice that you translated all the files. Thanks! I think we should not capitalize any letters in "wireless debugging", because this is how it is spelled in system strings. "Wi-Fi" is better like you suggested, because it is like this in system strings.

image

app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
@igormiguell
Copy link
Contributor Author

fixed

@Smooth-E
Copy link
Owner

Smooth-E commented Jul 17, 2024

Finally got the time to work on this project for a bit. I've applied your translations to the screenshots and readme media. What do you think, are they ok? Will wait until you agree, will create a changelog and then release 1.3 with your translations.

@Smooth-E
Copy link
Owner

@igormiguell please, also translate the changelog for version 3. Apparently, I forgot to push it, when I released 1.2 😅

@igormiguell
Copy link
Contributor Author

Finally got the time to work on this project for a bit. I've applied your translations to the screenshots and readme media. What do you think, are they ok? Will wait until you agree, will create a changelog and then release 1.3 with your translations.

The screenshots are great, ready to merge.

@igormiguell
Copy link
Contributor Author

@igormiguell please, also translate the changelog for version 3. Apparently, I forgot to push it, when I released 1.2 😅

translated

@Smooth-E Smooth-E merged commit 8d5d26f into Smooth-E:main Jul 17, 2024
@Smooth-E
Copy link
Owner

Done. Thanks for your contribution! Expect the new release soon.

@lucasmz-dev
Copy link

lucasmz-dev commented Jul 17, 2024

As a confirmation, this looks alright :P
- PT-BR speaker

@Smooth-E
Copy link
Owner

As a confirmation, this looks alright :P PT-BR speaker

Thanks for reviewing 🤩

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.

3 participants