Skip to content

Conversation

ceb10n
Copy link
Contributor

@ceb10n ceb10n commented Jan 13, 2025

No description provided.

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

@svlandeg svlandeg added lang-pt Portuguese translations awaiting-review labels Jan 13, 2025
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.

Excellent, soon we will have the whole documentation translated.

@cassiobotaro, do you have time to review this good work and provide your approval?

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.

very good!! I only pointed out 3 corrections. great work 🚀

@ceb10n
Copy link
Contributor Author

ceb10n commented Jan 15, 2025

very good!! I only pointed out 3 corrections. great work 🚀

Thank you for the review. I had no idea about the comma with etc 🤯

Fixed it all 😄

Copy link
Contributor

@nazarepiedady
Copy link
Contributor

@vitumenezes, could you provide your approval to allow the pull request to be merged?

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.

LGTM!

@ceb10n
Copy link
Contributor Author

ceb10n commented Jan 15, 2025

This was the last doc that was missing pt translations! 🎊🎉🥂🍾

#9187 (comment)

@alejsdev alejsdev merged commit 16199c4 into fastapi:master Jan 15, 2025
56 checks passed
@alejsdev
Copy link
Member

Thank you! @ceb10n 🚀 💯
And thanks for the review @vitumenezes 🔍

s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull request Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-2 lang-all Translations lang-pt Portuguese translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants