Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

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

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

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

Conversation

@ceb10n
Copy link
Contributor

No description provided.

@github-actionsgithub-actionsbot added the lang-allTranslations labelJan 13, 2025
@github-actions
Copy link
Contributor

@svlandegsvlandeg added lang-ptPortuguese translations awaiting-review labelsJan 13, 2025
Copy link
Contributor

@nazarepiedadynazarepiedady 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?

ceb10n reacted with heart emoji
Copy link
Contributor

@vitumenezesvitumenezes 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 reacted with heart emoji
@ceb10n
Copy link
ContributorAuthor

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 😄

vitumenezes reacted with hooray emojinazarepiedady and vitumenezes reacted with heart emoji

@github-actions
Copy link
Contributor

@nazarepiedady
Copy link
Contributor

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

vitumenezes reacted with thumbs up emoji

Copy link
Contributor

@vitumenezesvitumenezes 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 reacted with heart emoji
@ceb10n
Copy link
ContributorAuthor

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

#9187 (comment)

vitumenezes and alejsdev reacted with hooray emoji

@alejsdevalejsdev merged commit16199c4 intofastapi:masterJan 15, 2025
56 checks passed
@alejsdev
Copy link
Member

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

ceb10n, vitumenezes, and nazarepiedady reacted with hooray emoji

s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull requestJan 23, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

2 more reviewers

@vitumenezesvitumenezesvitumenezes approved these changes

@nazarepiedadynazarepiedadynazarepiedady approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved-2lang-allTranslationslang-ptPortuguese translations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ceb10n@nazarepiedady@alejsdev@vitumenezes@svlandeg

Comments

[8]ページ先頭
©2009-2026 Movatter.jp