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

🌐 Add Korean translation for docs/ko/docs/python-types.md #2267

Merged
merged 30 commits into from Jan 22, 2024

Conversation

jrim
Copy link
Contributor

@jrim jrim commented Oct 30, 2020

Korean translated 'python-types.md' #2017

@codecov
Copy link

codecov bot commented Oct 30, 2020

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cf73051) 100.00% compared to head (d2a0e65) 100.00%.
Report is 1086 commits behind head on master.

❗ Current head d2a0e65 differs from pull request most recent head 655e68d. Consider uploading reports for the commit 655e68d to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##            master     #2267     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files          540       240    -300     
  Lines        13969      7101   -6868     
===========================================
- Hits         13969      7101   -6868     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jrim jrim changed the title Add Korean Translation for Python-types #2017 Add Korean Translation for Python-types Oct 30, 2020
@github-actions
Copy link
Contributor

📝 Docs preview for commit eb3ac10 at: https://5f9bd3234b8985d7301987f1--fastapi.netlify.app

@jrim jrim changed the title Add Korean Translation for Python-types Add Korean Translation for python-types.md Oct 30, 2020
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.

I updated #2017 a few days ago. I want you to follow it.
Sorry for the late update.😭

docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
jrim and others added 4 commits November 6, 2020 21:08
Co-authored-by: hard_coder <rurouni24@gmail.com>
Co-authored-by: hard_coder <rurouni24@gmail.com>
Co-authored-by: hard_coder <rurouni24@gmail.com>
Co-authored-by: hard_coder <rurouni24@gmail.com>
@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2020

📝 Docs preview for commit d2a0e65 at: https://5fa53daa31aca1ff592e3b27--fastapi.netlify.app

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.

We are almost there!😎

But there's one more thing... to do. Please replace "에디터" with "편집기". It is also to follow python's official Korean docs.

docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
Copy link
Contributor

@SueNaEunYang SueNaEunYang left a comment

Choose a reason for hiding this comment

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

Hello, thank you for your efforts.
I added some comments here.
I hope they can be of any help for you :)

docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
```

!!! info
Pydantic<에 대해 더 배우고 싶다면 <a href="https://pydantic-docs.helpmanual.io/" class="external-link" target="_blank">공식 문서</a>를 참고하세요.</a>
Copy link
Contributor

Choose a reason for hiding this comment

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

How about "참고하십시오/확인해보십시오." rather than "참고하세요." ?

docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
jrim and others added 11 commits May 4, 2021 21:04
Co-authored-by: SueNaeunYang <artisticsue@naver.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
jrim and others added 8 commits May 4, 2021 21:06
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: SueNaeunYang <artisticsue@naver.com>
Co-authored-by: SueNaeunYang <artisticsue@naver.com>
Co-authored-by: SueNaeunYang <artisticsue@naver.com>
Co-authored-by: SueNaeunYang <artisticsue@naver.com>
Co-authored-by: SueNaeunYang <artisticsue@naver.com>
Co-authored-by: SueNaeunYang <artisticsue@naver.com>
Co-authored-by: Spike <rurouni24@gmail.com>
@jrim
Copy link
Contributor Author

jrim commented May 4, 2021

Thanks for reviewing y'all

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.

I reviewed some lines more, like

!!! tip

to

!!! tip "팁"

Please see reviews and modify what you can accept.
I hope this PR merged soon:D

docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
docs/ko/docs/python-types.md Outdated Show resolved Hide resolved
jrim and others added 4 commits May 11, 2021 14:07
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
Co-authored-by: Spike <rurouni24@gmail.com>
joonas-yoon added a commit to joonas-yoon/fastapi that referenced this pull request Aug 6, 2022
@tiangolo tiangolo changed the title Add Korean Translation for python-types.md 🌐 Add Korean Translation for docs/ko/docs/python-types.md Jun 26, 2023
@tiangolo tiangolo changed the title 🌐 Add Korean Translation for docs/ko/docs/python-types.md 🌐 Add Korean translation for docs/ko/docs/python-types.md Jun 26, 2023
@tiangolo
Copy link
Owner

📝 Docs preview for commit 655e68d at: https://6499cd5a4a964606cbd74e3b--fastapi.netlify.app

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.

It has been forgotten for a long time, so the Type Hints with Metadata Annotations section isn't reflected in this PR. Despite the problem, because it is well-translated, I approve it and will create a new pull request to add the section.

I'm sorry for not getting back to it sooner. 🙇🏻

@alejsdev alejsdev merged commit 2a8f8d1 into tiangolo:master Jan 22, 2024
30 checks passed
@alejsdev
Copy link
Collaborator

Thank you! @jrim 🐱

And thanks for the reviews @hard-coders and @SueNaEunYang 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants