Skip to content

🌐 Update Portuguese translations #13156

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 4 commits into from
Jan 9, 2025
Merged

🌐 Update Portuguese translations #13156

merged 4 commits into from
Jan 9, 2025

Conversation

nillvitor
Copy link
Contributor

Update Portuguese translation for the files:

  • docs/pt/docs/tutorial/index.md
  • docs/pt/docs/tutorial/first-steps.md
  • docs/pt/docs/deployment/server-workers.md
  • docs/pt/docs/deployment/manually.md

@github-actions github-actions bot added the lang-all Translations label Jan 4, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 4, 2025

📝 Docs preview for commit 5d70050 at: https://a7df8269.fastapitiangolo.pages.dev

Modified Pages

@github-actions github-actions bot added the docs Documentation about how to use FastAPI label Jan 4, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 4, 2025

📝 Docs preview for commit 48f8370 at: https://b3eb25f0.fastapitiangolo.pages.dev

Modified Pages

@alejsdev alejsdev changed the title Update Portuguese translation 🌐 Update Portuguese translations Jan 5, 2025
@alejsdev alejsdev added lang-pt Portuguese translations awaiting-review and removed docs Documentation about how to use FastAPI labels Jan 5, 2025
nazarepiedady
nazarepiedady approved these changes Jan 5, 2025
View reviewed changes
Copy link
Contributor

@nazarepiedady nazarepiedady 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 good, but could you update your branch and pull request changes that can not cause conflicts?

vitumenezes
vitumenezes reviewed Jan 7, 2025
View reviewed changes
Copy link
Contributor

@vitumenezes vitumenezes left a comment

Choose a reason for hiding this comment

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

Just some fixes by adding inline code. And there is also the problem of conflict.

@github-actions github-actions bot added the docs Documentation about how to use FastAPI label Jan 7, 2025
@alejsdev alejsdev removed the docs Documentation about how to use FastAPI label Jan 7, 2025
@alejsdev
Copy link
Member

alejsdev commented Jan 7, 2025

Hi @nillvitor, thanks for your interest in contributing but the Peewee file is no longer needed.
Could you please update this PR to avoid this conflict? Thanks 🙌

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 7, 2025

📝 Docs preview for commit 5d70050 at: https://e175473c.fastapitiangolo.pages.dev

Modified Pages

@nillvitor
Copy link
Contributor Author

Hi @alejsdev, I apologize for the inconvenience. I've corrected the branch and resolved the issues. Thank you for your patience!

@github-actions github-actions bot removed the waiting label Jan 7, 2025
@alejsdev alejsdev requested a review from vitumenezes January 8, 2025 11:10
ceb10n
ceb10n approved these changes Jan 9, 2025
View reviewed changes
Copy link
Contributor

@ceb10n ceb10n left a comment

Choose a reason for hiding this comment

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

✨

@alejsdev alejsdev merged commit 9d293b7 into fastapi:master Jan 9, 2025
103 checks passed
@alejsdev
Copy link
Member

alejsdev commented Jan 9, 2025

Thank you! @nillvitor 🚀 ✨

s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull request Jan 23, 2025
@nillvitor @s-rigaud
🌐 Update Portuguese translations (fastapi#13156)
35a80ea
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vitumenezes vitumenezes Awaiting requested review from vitumenezes

2 more reviewers

@ceb10n ceb10n ceb10n approved these changes

@nazarepiedady nazarepiedady nazarepiedady approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
approved-2 lang-all Translations lang-pt Portuguese translations
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@nillvitor @alejsdev @ceb10n @vitumenezes @nazarepiedady