Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🌐 Fix Korean translation for docs/ko/docs/tutorial/index.md #4193

Merged
merged 12 commits into from
Dec 8, 2021
Merged

🌐 Fix Korean translation for docs/ko/docs/tutorial/index.md #4193

merged 12 commits into from
Dec 8, 2021

Conversation

kimjaeyoonn
Copy link
Contributor

@kimjaeyoonn kimjaeyoonn commented Nov 19, 2021

Hello. This PR is an updated document to correct some inconveniences in #2317.

@hard-coders
Please check a few updated sentences, and if you find it appropriate, please leave a positive comment.
Additionally, it would be much appreciated if you could suggest a merge to tiangolo.

This PR complements ko/docs/tutorial/index.md
related : #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.

Appreciate your works! 🚀
It's better to read the docs.

Copy link
Sponsor Contributor

@hard-coders hard-coders left a comment

Choose a reason for hiding this comment

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

Thanks for your feedback! It's more natural.
I left some of my opinions. Would you mind checking and telling me about your thought?

Comment on lines 5 to 7
각 섹션은 이전 섹션을 기반으로 하여 점진적으로 만들어 졌지만, 주제에 따라 다르게 구성되었기 때문에 특정 API 요구사항을 해결하기 위해서라면 어느 특정 항목으로던지 직접 이동할 수 있습니다.

또한 향후 참조가 될 수 있도록 만들어졌습니다.
추가적으로 이는 향후 참조가 될 수 있도록 만들어졌습니다.
Copy link
Sponsor Contributor

Choose a reason for hiding this comment

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

  • 기반해서 -> 기반으로 하여: 둘이 완전히 동일한 뜻이므로 굳이 풀어서 사용할 이유는 없어 보입니다.
  • 또한 -> 추각적으로 이는: 너무 직역체입니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for posting a reply!!
The documentation has been updated to reflect your additional comments.
I hope the updated documents will be merged🚀

Copy link
Sponsor Contributor

@hard-coders hard-coders left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@tiangolo tiangolo changed the title 🌐 Fix Korean translation for Tutorial - intro 🌐 Fix Korean translation for docs/ko/docs/tutorial/index.md Dec 8, 2021
@tiangolo tiangolo added approved-2 lang-all Translations lang-ko Korean translations labels Dec 8, 2021
@tiangolo
Copy link
Owner

tiangolo commented Dec 8, 2021

Awesome, thanks for the contribution @kimjaeyoonn! 🙇

And thanks for the reviews @hard-coders, @kty4119, @DevDae! 👏

@tiangolo tiangolo enabled auto-merge (squash) December 8, 2021 15:27
@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2021

📝 Docs preview for commit a94ed56 at: https://61b0cf5b48f2a82f86cb214c--fastapi.netlify.app

@codecov
Copy link

codecov bot commented Dec 8, 2021

Codecov Report

Merging #4193 (a94ed56) into master (58ab733) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #4193   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          409       409           
  Lines        10264     10264           
=========================================
  Hits         10264     10264           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58ab733...a94ed56. Read the comment docs.

@tiangolo tiangolo merged commit dabb489 into tiangolo:master Dec 8, 2021
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.

None yet

5 participants