8000 🌐 Add Korean translation for `docs/ko/docs/help-fastapi.md` by kty4119 · Pull Request #4139 · fastapi/fastapi · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@kty4119
Copy link
Contributor
@kty4119 kty4119 commented Nov 4, 2021
  • Add Korean translation for docs/ko/docs/help-fastapi.md
  • Related [FEATURE] Korean translations [FEATURE] Korean translations [FEATURE] Korean translations Korean translations #2017

Copy link
Contributor
@DevDae DevDae left a comment

Choose a reason for hiding this comment

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

8000

Appreciate your hard works! 🙏
I will wait your comments, thank you!

@tiangolo tiangolo added awaiting-review lang-all Translations lang-ko Korean translations labels Dec 8, 2021
@github-actions github-actions bot mentioned this pull request Dec 8, 2021
joonas-yoon added a commit to joonas-yoon/fastapi that referenced this pull request Aug 6, 2022
@tiangolo
Copy link
Member

📝 Docs preview for commit b033d64 at: https://6499e34684c0620543c16807--fastapi.netlify.app

@tiangolo tiangolo requested a review from DevDae January 15, 2024 15:48
@tiangolo
Copy link
Member

Thank you! I'm still waiting for others to review this to be able to merge it, meanwhile, maybe you could help me review some of the other Korean translations, that way if they are approved by each other, I could merge them. 🤓

https://github.com/tiangolo/fastapi/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-desc+label%3Alang-ko+label%3Aawaiting-review

Maybe @DevDae could review it again after the requested changes? 🙏

8000

@alejsdev alejsdev merged commit 5a29797 into fastapi:master Apr 2, 2024
@alejsdev
Copy link
Member
alejsdev commented Apr 2, 2024

Thanks for the help! @kty4119 🤓 💯

And thanks for the reviews @DevDae @junah201 🔍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-1 lang-all Translations lang-ko Korean translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

0