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/security/first-steps.md #5643

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

KdHyeon0661
Copy link
Contributor

  • Add Korean translation for docs/ko/docs/tutorial/security/first-steps.md
  • Related [FEATURE] Korean translations Korean translations #2017

@github-actions
Copy link
Contributor

📝 Docs preview for commit 3342463 at: https://63725300a644e402e06ad807--fastapi.netlify.app

@tiangolo tiangolo added lang-all Translations awaiting-review lang-ko Korean translations labels Nov 14, 2022
@github-actions github-actions bot mentioned this pull request Nov 14, 2022
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.

great works 👍

!!! 정보
먼저 <a href="https://andrew-d.github.io/python-multipart/" class="external-link" target="_blank">`python-multipart`</a>를 설치합니다.

E.g. `pip install python-multipart`.
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
E.g. `pip install python-multipart`.
명령어 예시: `pip install python-multipart`

Comment on lines 170 to 172
'Authorization' 헤더가 표시되지 않거나 값에 'Bearer' 토큰이 없으면 401 상태 코드 오류('UNAUTHORIZED')로 직접 응답합니다.

오류를 반환하기 위해 토큰이 있는지 확인할 필요조차 없습니다. 함수가 실행되면 해당 토큰에 'str'이 있음을 확신할 수 있습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

missed format

Suggested change
'Authorization' 헤더가 표시되지 않거나 값에 'Bearer' 토큰이 없으면 401 상태 코드 오류('UNAUTHORIZED')로 직접 응답합니다.
오류를 반환하기 위해 토큰이 있는지 확인할 필요조차 없습니다. 함수가 실행되면 해당 토큰에 'str'이 있음을 확신할 수 있습니다.
`Authorization` 헤더가 표시되지 않거나 값에 `Bearer` 토큰이 없으면 401 상태 코드 오류(`UNAUTHORIZED`)로 직접 응답합니다.
오류를 반환하기 위해 토큰이 있는지 확인할 필요조차 없습니다. 함수가 실행되면 해당 토큰에 `str`이 있음을 확신할 수 있습니다.

{!../../../docs_src/security/tutorial001.py!}
```

이 종속성은 *경로 작동 함수*의 'token' 매개변수에 할당된 'str'을 제공합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
이 종속성은 *경로 작동 함수*'token' 매개변수에 할당된 'str'을 제공합니다.
이 종속성은 *경로 작동 함수*`token` 매개변수에 할당된 `str`을 제공합니다.

* 사용자가 프런트엔드를 클릭하여 프런트엔드 웹 앱의 다른 섹션으로 이동합니다.
* 프론트엔드는 API에서 더 많은 데이터를 가져와야 합니다.
* 그러나 특정 엔드포인트에 대한 인증이 필요합니다.
* 따라서 API로 인증하기 위해 'Bearer' 값에 토큰을 더한 'Authorization' 헤더를 보냅니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* 따라서 API로 인증하기 위해 'Bearer' 값에 토큰을 더한 'Authorization' 헤더를 보냅니다.
* 따라서 API로 인증하기 위해 `Bearer` 값에 토큰을 더한 `Authorization` 헤더를 보냅니다.

Comment on lines 76 to 80
## `패스워드` 흐름

이제 조금 돌아가서 그것이 무엇인지 이해합시다.

`패스워드` "흐름"은 보안 및 인증을 처리하기 위해 OAuth2에 정의된 방식("흐름") 중 하나입니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is hard. you keep it to be english name as 패스워드 and translated Flow into 흐름 (not 플로우).

In my opinion, what about one of this in following options:

  • 패스워드 플로우
  • 비밀번호 흐름

@github-actions
Copy link
Contributor

📝 Docs preview for commit de6c1d4 at: https://6377561492b0af2ea2811ebb--fastapi.netlify.app

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2022

📝 Docs preview for commit f50dc32 at: https://63898f6da62879358786a5a0--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit 647233e at: https://639ce5dfc1082f0408d1c885--fastapi.netlify.app

@tiangolo
Copy link
Owner

📝 Docs preview for commit 7f44a43 at: https://649a1ad1da0f49117d6c453d--fastapi.netlify.app

@tiangolo
Copy link
Owner

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

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

3 participants