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

feat/nginx-mailcow_brazilian-translations #6048

Merged

Conversation

airon-assustadus
Copy link
Contributor

Contribution Guidelines

What does this PR include?

Adding some brazilian translations that were missing

Short Description

Adding some brazilian translations that were missing

Affected Containers

  • nginx-mailcow

Did you run tests?

Yes, I did

What did you tested?

I open some screen that wasn't translated property

What were the final results? (Awaited, got)

Everything is OK

# What
- Adding some brazilian translations that were missing
@MAGICCC
Copy link
Member

MAGICCC commented Aug 24, 2024

Nice, but why did you remove some strings?
Btw you may check out https://translate.mailcow.email/engage/mailcow-dockerized/

@DerLinkman DerLinkman requested a review from MAGICCC August 27, 2024 21:12
@airon-assustadus
Copy link
Contributor Author

Hello @MAGICCC

Sorry for the delay,
I haven't deleted them, I just organized it alphabetically.

Best Regards

@MAGICCC MAGICCC merged commit 22f7f61 into mailcow:staging Sep 5, 2024
2 checks passed
@MAGICCC
Copy link
Member

MAGICCC commented Sep 5, 2024

Ah ok, thanks!

DerLinkman pushed a commit that referenced this pull request Nov 15, 2024
# What
- Adding some brazilian translations that were missing

Co-authored-by: Airon Teixeira <airon@ymail.com>
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