Thanks to visit codestin.com
Credit goes to github.com

Skip to content

🌐 Add Portuguese translation for docs/pt/docs/tutorial/security/oauth2-jwt.md#13205

Merged
alejsdev merged 3 commits into
fastapi:masterfrom
ceb10n:features/translate-oauth-2-jwt-pt
Jan 15, 2025
Merged

🌐 Add Portuguese translation for docs/pt/docs/tutorial/security/oauth2-jwt.md#13205
alejsdev merged 3 commits into
fastapi:masterfrom
ceb10n:features/translate-oauth-2-jwt-pt

Conversation

@ceb10n

@ceb10n ceb10n commented Jan 13, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the lang-all Translations label Jan 13, 2025
@github-actions

github-actions Bot commented Jan 13, 2025

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 349a7a8 at: https://d43382b6.fastapitiangolo.pages.dev

Modified Pages

@svlandeg svlandeg added lang-pt Portuguese translations awaiting-review labels Jan 13, 2025
nazarepiedady
nazarepiedady approved these changes Jan 14, 2025
View reviewed changes

@nazarepiedady nazarepiedady left a comment

Copy link
Copy Markdown
Contributor

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?

vitumenezes
vitumenezes reviewed Jan 14, 2025
View reviewed changes

@vitumenezes vitumenezes left a comment

Copy link
Copy Markdown
Contributor

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 🚀

Comment thread docs/pt/docs/tutorial/security/oauth2-jwt.md Outdated
Comment thread docs/pt/docs/tutorial/security/oauth2-jwt.md Outdated
Comment thread docs/pt/docs/tutorial/security/oauth2-jwt.md Outdated
@ceb10n

ceb10n commented Jan 15, 2025

Copy link
Copy Markdown
Contributor Author

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 😄

@github-actions

github-actions Bot commented Jan 15, 2025

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 45b76dd at: https://0e24ffc4.fastapitiangolo.pages.dev

Modified Pages

@nazarepiedady

nazarepiedady commented Jan 15, 2025

Copy link
Copy Markdown
Contributor

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

vitumenezes
vitumenezes approved these changes Jan 15, 2025
View reviewed changes

@vitumenezes vitumenezes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@ceb10n

ceb10n commented Jan 15, 2025

Copy link
Copy Markdown
Contributor Author

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

#9187 (comment)

@alejsdev alejsdev merged commit 16199c4 into fastapi:master Jan 15, 2025
@alejsdev

alejsdev commented Jan 15, 2025

Copy link
Copy Markdown
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
@ceb10n @s-rigaud
🌐 Add Portuguese translation for `docs/pt/docs/tutorial/security/oaut…
acd5fed 
…h2-jwt.md` (fastapi#13205)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@vitumenezes vitumenezes vitumenezes 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

@ceb10n @nazarepiedady @alejsdev @vitumenezes @svlandeg