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 PT-BR translation #6345

Merged
merged 3 commits into from
Apr 19, 2024
Merged

Update PT-BR translation #6345

merged 3 commits into from
Apr 19, 2024

Conversation

PedroPMS
Copy link
Contributor

Closes #6333

Changes proposed in this pull request:

I'm sending the first contribution to the Brazilian Portuguese translation!

How to test the feature manually:

  1. Select Portuguese Brazilian lanaguage
  2. Use the system lol

Pull request checklist:

  • clear commit messages
  • code manually tested
  • unit tests written (optional if too hard)
  • documentation updated

Additional information can be found in the documentation.

@PedroPMS
Copy link
Contributor Author

@NicolasPereira review pls <3

Copy link
Contributor

@NicolasPereira NicolasPereira left a comment

Choose a reason for hiding this comment

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

LGTM

app/i18n/pt-br/admin.php Outdated Show resolved Hide resolved
app/i18n/pt-br/conf.php Outdated Show resolved Hide resolved
app/i18n/pt-br/conf.php Outdated Show resolved Hide resolved
app/i18n/pt-br/conf.php Outdated Show resolved Hide resolved
app/i18n/pt-br/conf.php Outdated Show resolved Hide resolved
app/i18n/pt-br/conf.php Outdated Show resolved Hide resolved
app/i18n/pt-br/conf.php Outdated Show resolved Hide resolved
@NicolasPereira
Copy link
Contributor

@math-GH FYI

app/i18n/pt-br/admin.php Outdated Show resolved Hide resolved
Copy link
Contributor

@NicolasPereira NicolasPereira left a comment

Choose a reason for hiding this comment

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

LGTM
Thanks for your contribution

app/i18n/pt-br/sub.php Outdated Show resolved Hide resolved
@math-GH
Copy link
Contributor

math-GH commented Apr 19, 2024

Wow, that is amazing! That was very fast and a great team work of you!

@PedroPMS Welcome :) Please add yourself on our contributors list Credits.md :)

@Alkarex Alkarex added this to the 1.24.0 milestone Apr 19, 2024
@Alkarex Alkarex added the I18n 🌍 Translations label Apr 19, 2024
@Alkarex Alkarex merged commit dcd3b80 into FreshRSS:edge Apr 19, 2024
2 checks passed
@PedroPMS
Copy link
Contributor Author

Wow, that is amazing! That was very fast and a great team work of you!

@PedroPMS Welcome :) Please add yourself on our contributors list Credits.md :)

#6345

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

Successfully merging this pull request may close these issues.

[i18n] Translation Brazil Portuguese (PT_BR) 🇧🇷 needs translators
6 participants