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.

Sign up for GitHub

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

Jump to bottom

🌐 Add Korean translation for Tutorial - JSON Compatible Encoder #3152

Merged

tiangolo merged 2 commits into fastapi:master from NEONKID:ko-doc/json-compatiable-encoder

Dec 8, 2021

Contributor

NEONKID commented May 1, 2021 •

edited

Loading

Hello there.

Translated by referring to the previous PR. If you have any problems such as mistranslations, please feel free to leave your comments here.

This PR translates tutorial/encoder.md in Korean.

related: #2017


          translate tutorial/encoder.md

0ca4843

tiangolo added awaiting-review lang-all lang-ko labels

codecov bot commented May 1, 2021 •

edited

Loading

Codecov Report

Merging #3152 (0ca4843) into master (ce0ec06) will not change coverage.
The diff coverage is n/a.

❗ Current head 0ca4843 differs from pull request most recent head d1fa3ae. Consider uploading reports for the commit d1fa3ae to get more accurate results

@@            Coverage Diff            @@
##            master     #3152   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          243       243           
  Lines         7419      7419           
=========================================
  Hits          7419      7419

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 2252837...d1fa3ae. Read the comment docs.

Contributor

github-actions bot commented May 1, 2021

📝 Docs preview for commit 0ca4843 at: https://608d280e7010f90923b80230--fastapi.netlify.app

hard-coders reviewed

View reviewed changes

docs/ko/docs/tutorial/encoder.md Outdated

		@@ -0,0 +1,34 @@
		# JSON Compatible 인코더

Contributor

hard-coders May 4, 2021

You missed Compatible.

docs/ko/docs/tutorial/encoder.md Outdated


		사이즈가 큰 문자열 형태의 JSON 형식(문자열)의 데이터가 들어있을 땐 `str`로 반환하지 않습니다. JSON과 모두 호환되는 값과 하위 값이 있는 Python 표준 데이터 구조 (예: `dict`)를 반환합니다.

		!!! note

Contributor

hard-coders May 4, 2021

Suggested change

      
            !!! note
          
            !!! note "참고"

docs/ko/docs/tutorial/encoder.md Outdated


		이를 위해 `jsonable_encoder` 를 사용할 수 있습니다.

		Pydantic 모델과 같은 객체를 받고 JSON 호환 가능한 데이터로 반환합니다:

Contributor

hard-coders May 4, 2021

Suggested change

      
            Pydantic 모델과 같은 객체를 받고 JSON 호환 가능한 데이터로 반환합니다:
          
            Pydantic 모델과 같은 객체를 받고 JSON 호환 가능한 버전으로 반환합니다:

It's not a big deal; the original sentence used the term version, not data however.

It's good to keep the intention of the author as you can possible.

docs/ko/docs/tutorial/encoder.md Outdated

+              사이즈가 큰 문자열 형태의 JSON 형식(문자열)의 데이터가 들어있을 땐 `str`로 반환하지 않습니다. JSON과 모두 호환되는 값과 하위 값이 있는 Python 표준 데이터 구조 (예: `dict`)를 반환합니다.
+              !!! note
+              `jsonable_encoder` 은 실제로 **FastAPI** 에서 내부적으로 데이터를 변환하는 데 사용하지만, 다른 많은 곳에서도 이는 유용합니다.

Contributor

hard-coders May 4, 2021

Again, for preserving the intention, don't break a sentence structure, please. 🤗

And I think "는" is proper than "은" where jsonable_encoder after.

peebbv6364 reviewed

View reviewed changes

peebbv6364 left a comment

I propose some modifications. However, you can reject the offer because it is simply an expression of my thoughts.

docs/ko/docs/tutorial/encoder.md Outdated


		이렇게 호출한 결과는 파이썬 표준인 <a href="https://docs.python.org/3/library/json.html#json.dumps" class="external-link" target="_blank">`json.dumps()`</a>로 인코딩 할 수 있습니다.

		사이즈가 큰 문자열 형태의 JSON 형식(문자열)의 데이터가 들어있을 땐 `str`로 반환하지 않습니다. JSON과 모두 호환되는 값과 하위 값이 있는 Python 표준 데이터 구조 (예: `dict`)를 반환합니다.

peebbv6364 May 4, 2021

Suggested change

      
            사이즈가 큰 문자열 형태의 JSON 형식(문자열)의 데이터가 들어있을 땐 `str`로 반환하지 않습니다. JSON과 모두 호환되는 값과 하위 값이 있는 Python 표준 데이터 구조 (예: `dict`)를 반환합니다.
          
            길이가 긴 문자열 형태의 JSON 형식(문자열)의 데이터가 들어있는 상황에서는 `str`로 반환하지 않습니다. JSON과 모두 호환되는 값과 하위 값이 있는 Python 표준 데이터 구조 (예: `dict`)를 반환합니다.

docs/ko/docs/tutorial/encoder.md Outdated

+              사이즈가 큰 문자열 형태의 JSON 형식(문자열)의 데이터가 들어있을 땐 `str`로 반환하지 않습니다. JSON과 모두 호환되는 값과 하위 값이 있는 Python 표준 데이터 구조 (예: `dict`)를 반환합니다.
+              !!! note
+              `jsonable_encoder` 은 실제로 **FastAPI** 에서 내부적으로 데이터를 변환하는 데 사용하지만, 다른 많은 곳에서도 이는 유용합니다.

peebbv6364 May 4, 2021

Suggested change

      
            `jsonable_encoder` 은 실제로 **FastAPI** 에서 내부적으로 데이터를 변환하는 데 사용하지만, 다른 많은 곳에서도 이는 유용합니다.
          
            실제로 `jsonable_encoder`는 **FastAPI** 에서 내부적으로 데이터를 변환하는 데 사용되지만, 다른 많은 곳에서도 유용하게 사용됩니다.

docs/ko/docs/tutorial/encoder.md Outdated


		사이즈가 큰 문자열 형태의 JSON 형식(문자열)의 데이터가 들어있을 땐 `str`로 반환하지 않습니다. JSON과 모두 호환되는 값과 하위 값이 있는 Python 표준 데이터 구조 (예: `dict`)를 반환합니다.

		!!! note

peebbv6364 May 4, 2021

Suggested change

      
            !!! note
          
            !!! 참고

Contributor

hard-coders May 4, 2021

We have to write like:

Suggested change

      
            !!! note
          
            !!! note "참고"

docs/ko/docs/tutorial/encoder.md Outdated


		## `jsonable_encoder` 사용

		JSON 호환 가능 데이터만 수신하는 `fake_db` 데이터베이스가 있다고 가정해보겠습니다.

peebbv6364 May 4, 2021

Suggested change

      
            JSON 호환 가능 데이터만 수신하는 `fake_db` 데이터베이스가 있다고 가정해보겠습니다.
          
            JSON 호환 가능 데이터만 수신하는 `fake_db` 데이터베이스가 존재한다고 가정하겠습니다.

docs/ko/docs/tutorial/encoder.md Outdated


		JSON 호환 가능 데이터만 수신하는 `fake_db` 데이터베이스가 있다고 가정해보겠습니다.

		예를 들면, `datetime` 객체는 JSON과 호환되는 데이터가 아니므로 이 데이터는 받지 않습니다.

peebbv6364 May 4, 2021

Suggested change

      
            예를 들면, `datetime` 객체는 JSON과 호환되는 데이터가 아니므로 이 데이터는 받지 않습니다.
          
            예를 들면, `datetime` 객체는 JSON과 호환되는 데이터가 아니므로 이 데이터는 받아들여지지 않습니다.


          Fix translate sentences..

d1fa3ae

Contributor Author

NEONKID commented May 4, 2021 •

edited

Loading

@hard-coders @peebbv6364
Thank you for reviewing. I have actively accepted and reflected your opinions.

hard-coders mentioned this pull request

Korean translations #2017

Closed

hard-coders approved these changes

View reviewed changes

Contributor

0417taehyun commented Nov 3, 2021

Can you approve the review?

peebbv6364 approved these changes

View reviewed changes

github-actions bot added approved-2 and removed awaiting-review labels

Member

tiangolo commented Dec 8, 2021

Thanks for your contribution @NEONKID! 🍰

And thanks for the reviews @hard-coders and @peebbv6364! 🙇

tiangolo merged commit 461e0d4 into fastapi:master

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

Labels

approved-2 lang-all lang-ko