8000 Fixing and adding missing pt-br translations by gtkacz · Pull Request #9535 · encode/django-rest-framework · GitHub
[go: up one dir, main page]
Skip to content
/ django-rest-framework Public
< 8000 /div>

Fixing and adding missing pt-br translations #9535

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 20, 2024
Merged

Fixing and adding missing pt-br translations #9535

merged 1 commit into from
Sep 20, 2024

Conversation

gtkacz
Copy link
Contributor
@gtkacz gtkacz commented Sep 15, 2024

Note: Before submitting a code change, please review our contributing guidelines.

Description

Fixing wrong translations and adding missing ones in PT-BR. Closes #9526.

@gtkacz gtkacz changed the title Fixing and adding pt-br translations Fixing and adding missing pt-br translations Sep 17, 2024
@tomchristie
Copy link
Member

We've generally been managing translations through transifex.
@encode/django-rest-framework should we just deal with them here instead?

@peterthomassen
Copy link
Contributor

I have no experience with Transifex, and thus no preference. What would be the pros and cons?

That said, it may be that this ended up here because the Transifex link in the docs was broken. Here's a fix: #9541

@TGoddessana
Copy link
Contributor

I recently applied to join the team to add a missing translation, but they haven't contacted me. Maybe it would be a good idea to handle the translation here?

@tomchristie
Copy link
Member
tomchristie commented Sep 20, 2024 •
edited
Loading

Cool, yep.

(Also, docs might need editing in this case)

@tomchristie tomchristie merged commit 81613c0 into encode:master Sep 20, 2024
8 checks passed
This was referenced Oct 22, 2024
Closed
Merged
Open
@browniebroke browniebroke added this to the 3.16 milestone Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomchristie tomchristie tomchristie approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.16
Development

Successfully merging this pull request may close these issues.

Incorrect Portuguese Transalation
5 participants
@gtkacz @tomchristie @peterthomassen @TGoddessana @browniebroke
0