Skip to content

🌐 Update Korean translation for docs/ko/docs/tutorial/security/simple-oauth2.md#13335

Merged
alejsdev merged 2 commits intofastapi:masterfrom
yes0ng:feature/translation-docs-ko-simple-oauth2
Feb 15, 2025
Merged

🌐 Update Korean translation for docs/ko/docs/tutorial/security/simple-oauth2.md#13335
alejsdev merged 2 commits intofastapi:masterfrom
yes0ng:feature/translation-docs-ko-simple-oauth2

Conversation

@yes0ng
Copy link
Contributor

@yes0ng yes0ng commented Feb 8, 2025

NOTE

As per the contributing guidelines, I modified the color of info boxes to display properly.

https://fastapi.tiangolo.com/contributing/#translation-specific-tips-and-guidelines

AS-IS

image

TO-BE

image

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

@timothy-jeong timothy-jeong left a comment

Choose a reason for hiding this comment

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

You've fixed the document formatting issues that had broken over time. Thank you!

@alejsdev alejsdev changed the title 🌐 Update Korean translation for docs/ko/docs/tutorial/security/simple-oauth2.md 🌐 Update Korean translation for docs/ko/docs/tutorial/security/simple-oauth2.md Feb 8, 2025
@alejsdev alejsdev added awaiting-review lang-ko Korean translations labels Feb 8, 2025
@yes0ng
Copy link
Contributor Author

yes0ng commented Feb 9, 2025

Thank you for your review @timothy-jeong

Copy link
Contributor

@11kkw 11kkw left a comment

Choose a reason for hiding this comment

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

Thank you for catching the formatting issues in the OAuth2 examples!

@alejsdev alejsdev merged commit 39b4692 into fastapi:master Feb 15, 2025
52 checks passed
@alejsdev
Copy link
Member

Thanks for your help! @yes0ng 🦫 ✨

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-ko Korean translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants