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/tutorial/dependencies/index.md #5605

Closed
wants to merge 11 commits into from

Conversation

KdHyeon0661
Copy link
Contributor

  • Add Korean translation for docs/ko/docs/tutorial/dependencies/index.md
  • Related [FEATURE] Korean translations Korean translations #2017

@github-actions
Copy link
Contributor

github-actions bot commented Nov 9, 2022

📝 Docs preview for commit d0daaff at: https://636beb817f6c0f05fb1b2c78--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit a4c7171 at: https://636cf466740e76271f77a48f--fastapi.netlify.app

@tiangolo tiangolo added lang-all Translations awaiting-review lang-ko Korean translations labels Nov 13, 2022
@github-actions github-actions bot mentioned this pull request Nov 13, 2022
Copy link

@jiyeonseo jiyeonseo left a comment

Choose a reason for hiding this comment

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

Great work!

I think it would be great to leave a comment on Korean translation discussion so everyone can check the progress of the translation and your excellent work!


## "의존성 주입"이 무엇입니까?

**"의존성 주입"**은 프로그래밍에서 코드(이 경우, 당신의 *경로 조작 함수*)에 작동하고 사용하는데 필요한 "의존성"들이 선언하는 방법입니다.

Choose a reason for hiding this comment

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

Suggested change
**"의존성 주입"**은 프로그래밍에서 코드(이 경우, 당신의 *경로 조작 함수*)에 작동하고 사용하는데 필요한 "의존성"들이 선언하는 방법입니다.
**"의존성 주입"**은 프로그래밍에서 코드(이 경우, 당신의 *경로 작동 함수*)에 작동하고 사용하는데 필요한 "의존성"들이 선언하는 방법입니다.

다른 변역들과 통일성이 있으면 좋을것 같습니다 :)
#3167 (comment)

Copy link
Contributor

Choose a reason for hiding this comment

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

also, you can refer #3167 (comment) (different comment with above one)


**2 lines**.

그리고 그것은 모든 *경로 조작 함수*이 가지고 있는 것과 같은 모양과 구조를 가지고 있습니다.

Choose a reason for hiding this comment

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

Suggested change
그리고 그것은 모든 *경로 조작 함수*이 가지고 있는 것과 같은 모양과 구조를 가지고 있습니다.
그리고 그것은 모든 *경로 작동 함수*이 가지고 있는 것과 같은 모양과 구조를 가지고 있습니다.


먼저 의존성에 초점을 맞춰봅시다.

그것은 단지 *경로 조작 함수*가 취할 수 있는 것과 동일한 모든 매개변수를 취할 수 있는 함수일 뿐입니다:

Choose a reason for hiding this comment

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

Suggested change
그것은 단지 *경로 조작 함수*가 취할 수 있는 것과 동일한 모든 매개변수를 취할 수 있는 함수일 뿐입니다:
그것은 단지 *경로 작동 함수*가 취할 수 있는 것과 동일한 모든 매개변수를 취할 수 있는 함수일 뿐입니다:


그리고 그것은 모든 *경로 조작 함수*이 가지고 있는 것과 같은 모양과 구조를 가지고 있습니다.

"데코레이터" 없이(`@app.get("/some-path")` 없이) *경로 조작 함수*로 생각할 수 있습니다.

Choose a reason for hiding this comment

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

Suggested change
"데코레이터" 없이(`@app.get("/some-path")` 없이) *경로 조작 함수*로 생각할 수 있습니다.
"데코레이터" 없이(`@app.get("/some-path")` 없이) *경로 작동 함수*로 생각할 수 있습니다.


### "의존자"에 의존성 명시하기

*경로 조작 함수* 매개변수와 함께 `Body`, `Query` 등을 사용하는 것과 같은 방식으로 새 매개변수와 함께 `Depends`를 사용합니다:

Choose a reason for hiding this comment

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

Suggested change
*경로 조작 함수* 매개변수와 함께 `Body`, `Query` 등을 사용하는 것과 같은 방식으로 새 매개변수와 함께 `Depends`를 사용합니다:
*경로 작동 함수* 매개변수와 함께 `Body`, `Query` 등을 사용하는 것과 같은 방식으로 새 매개변수와 함께 `Depends`를 사용합니다:


## `비동기` 혹은 `동기`

의존성들 또한 **FastAPI**(*경로 조작 함수*와 동일)에서도 호출되므로 함수를 정의하는 동안 동일한 규칙이 적용됩니다.

Choose a reason for hiding this comment

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

Suggested change
의존성들 또한 **FastAPI**(*경로 조작 함수*와 동일)에서도 호출되므로 함수를 정의하는 동안 동일한 규칙이 적용됩니다.
의존성들 또한 **FastAPI**(*경로 작동 함수*와 동일)에서도 호출되므로 함수를 정의하는 동안 동일한 규칙이 적용됩니다.


당신은 `async def` 혹은 일반 `def`를 사용할 수 있습니다.

그리고 일반 `def` *경로 조작 함수* 내부의 `async def`로 혹은 `async def` *경로 조작 함수* 내부의 `def`로 의존성을 선언하는 등이 있습니다.

Choose a reason for hiding this comment

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

Suggested change
그리고 일반 `def` *경로 조작 함수* 내부의 `async def`로 혹은 `async def` *경로 조작 함수* 내부의 `def`로 의존성을 선언하는 등이 있습니다.
그리고 일반 `def` *경로 작동 함수* 내부의 `async def`로 혹은 `async def` *경로 작동 함수* 내부의 `def`로 의존성을 선언하는 등이 있습니다.


이 방법으로 공유 코드를 한 번 작성하면 **FastAPI**가 *경로 함수*을 위해 이를 호출합니다.

!!! 체크

Choose a reason for hiding this comment

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

Suggested change
!!! 체크
!!! 확인

용어집에 있는 부분이라 통일되면 좋을 것 같습니다 :)
#3167 (comment)


문제 없습니다. **FastAPI**는 무엇을 할지 알고 있습니다.

!!! 노트

Choose a reason for hiding this comment

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

Suggested change
!!! 노트
!!! 정보

용어집에 있는 부분이라 통일되면 좋을 것 같습니다 :)
#3167 (comment)

문제 없습니다. **FastAPI**는 무엇을 할지 알고 있습니다.

!!! 노트
잘 모르겠다면, 문서의 [Async: *"In a hurry?"*](../../async.md){.internal-link target=_blank} 섹션에서 `async`과 `await`에 대해 확인하십시오.

Choose a reason for hiding this comment

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

Suggested change
잘 모르겠다면, 문서의 [Async: *"In a hurry?"*](../../async.md){.internal-link target=_blank} 섹션에서 `async`과 `await`에 대해 확인하십시오.
잘 모르겠다면, 문서의 [Async: *"In a hurry?"*](../../async.md){.internal-link target=_blank} 섹션에서 `async`과 `await`에 대해 확인할 수 있습니다.

다른 문장들과의 어투가 통일되면 더 읽기 좋을 것 같습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

모두 반영했습니다. 그리고 3.6은 지원이 끝나서 3.7로 교체해봤습니다.

@github-actions
Copy link
Contributor

📝 Docs preview for commit 25a94f0 at: https://6372567ea644e406956adb0b--fastapi.netlify.app

Copy link
Contributor

@joonas-yoon joonas-yoon left a comment

Choose a reason for hiding this comment

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

I left some comments :)


그리고 그 값들을 `dict` 형으로 반환합니다.

### `Depends` 암포트하기
Copy link
Contributor

Choose a reason for hiding this comment

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

here is typo 😝

Suggested change
### `Depends` 암포트하기
### `Depends` 불러오기

Comment on lines 46 to 48
끝입니다.

**2 lines**.
Copy link
Contributor

Choose a reason for hiding this comment

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

what about this.

Suggested change
끝입니다.
**2 lines**.
끝났습니다.
**단 두 줄로**.


"데코레이터" 없이(`@app.get("/some-path")` 없이) *경로 작동 함수*로 생각할 수 있습니다.

그리고 원한다면 언제든지 되돌릴 수 있습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Original sentence:

And it can return anything you want.

I don't think the meaning of return is to restore on here. It should be Return statement in programming.
So this is my suggestion:

Suggested change
그리고 원한다면 언제든지 되돌릴 수 있습니다.
그리고 원하는 타입대로 반환할 수도 있습니다.

@github-actions
Copy link
Contributor

📝 Docs preview for commit e0bf5d0 at: https://63774d3f24e44b1c8825d759--fastapi.netlify.app

@joonas-yoon
Copy link
Contributor

LGTM 👍

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2022

📝 Docs preview for commit 3a225cd at: https://63898f8924ec9c346627947d--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit 9bc6f49 at: https://639ce5c9a12b8e05b99de627--fastapi.netlify.app

@tiangolo
Copy link
Owner

📝 Docs preview for commit 49630fb at: https://649a1a917ce52313fb50ea45--fastapi.netlify.app

@tiangolo
Copy link
Owner

Thank you @KdHyeon0661! ☕

It seems this was handled in #10989, so I'll now close this one. But if you feel that should be updated, feel free to add new PRs. Thanks! 🍰

@tiangolo tiangolo closed this Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review lang-all Translations lang-ko Korean translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants