fastapi/fastapiPublic

NotificationsYou must be signed in to change notification settings
Fork8.7k
Star95.4k

🌐 Add Japanese translation for`docs/ja/docs/tutorial/response-model.md`#1938

Merged

alejsdev merged 29 commits intofastapi:masterfrom

SwftAlpc:translation/ja-tutorial-response-model-md

Jan 15, 2024

Merged

🌐 Add Japanese translation for`docs/ja/docs/tutorial/response-model.md`#1938
alejsdev merged 29 commits intofastapi:masterfrom
SwftAlpc:translation/ja-tutorial-response-model-md

Conversation

Copy link

Contributor

SwftAlpc commentedAug 20, 2020•
edited
Loading

This PR translates the tutorial/response-model.md to Japanese.
Please review tutorial/response-model.md.

Issue#1571 generates a new directory to support Japanese.
PR#1656 generates a new directory for tutorial.
I will fix dependency after#1571 &#1656 are merged.

ryuckeland others added14 commits

June 3, 2020 02:01

add ja and translate key feature

589de8c

translate opinions

2b5be36

translate by document upgrade

e983659

translate to end

ac92c7e

fix text

d9f234c

translate spoiler alert

babbd2f

Merge branch 'master' into feature/add-japanese-translation

9e2f971

fix translations and typo

d293652

Merge branch 'feature/add-japanese-translation' ofhttps://github.com…

7628a0b

…/ryuckel/fastapi into feature/add-japanese-translation

fix translations

75d6ccb

Add Japanese translation for tutorial/index.md

2289211

Fix tip word

cedd872

Improve translation

f5154a5

add tutorial/response-model.md

8182bcd

SwftAlpc mentioned this pull request

Aug 20, 2020

Japanese Translation#1572

Closed

Copy link

codecovbot commentedAug 20, 2020•
edited
Loading

Codecov Report

Merging#1938 intomaster willnot change coverage.
The diff coverage isn/a.

@@            Coverage Diff            @@##            master     #1938   +/-   ##=========================================  Coverage   100.00%   100.00%           =========================================  Files          239       239             Lines         7079      7079           =========================================  Hits          7079      7079

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatef495d98...f82f73d. Read thecomment docs.

Copy link

Contributor

github-actionsbot commentedAug 20, 2020

📝 Docs preview for commit8182bcd at:https://5f3ecaa40cca8f755feb49fd--fastapi.netlify.app

tokusumi suggested changes

Aug 21, 2020

View reviewed changes

docs/ja/docs/tutorial/response-model.md Outdated


		ここでは`UserIn`モデルを宣言しています。それは平文のパスワードが含まれています:

		```Python hl_lines="7 9"

Copy link

Contributor

tokusumiAug 21, 2020

Choose a reason for hiding this comment

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

hl_lines="9 11"

docs/ja/docs/tutorial/response-model.md Outdated


		そして、このモデルを使用して入力を宣言し、同じモデルを使って出力を宣言しています:

		```Python hl_lines="15 16"

Copy link

Contributor

tokusumiAug 21, 2020

Choose a reason for hiding this comment

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

hl_lines="17 18"

docs/ja/docs/tutorial/response-model.md Outdated


		代わりに、平文のパスワードを持つ入力モデルと、パスワードを持たない出力モデルを作成することができます:

		```Python hl_lines="7 9 14"

Copy link

Contributor

tokusumiAug 21, 2020

Choose a reason for hiding this comment

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

hl_lines="9 11 16"

docs/ja/docs/tutorial/response-model.md Outdated


		ここでは、path operation関数がパスワードを含む同じ入力ユーザーを返しているにもかかわらず:

		```Python hl_lines="22"

Copy link

Contributor

tokusumiAug 21, 2020

Choose a reason for hiding this comment

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

hl_lines="24"

docs/ja/docs/tutorial/response-model.md Outdated


		...`response_model`を`UserOut`と宣言したことで、パスワードが含まれていません:

		```Python hl_lines="20"

Copy link

Contributor

tokusumiAug 21, 2020

Choose a reason for hiding this comment

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

hl_lines="22"

docs/ja/docs/tutorial/response-model.md Outdated


		これは`response_model_include`や`response_mode_exclude`を使用していくつかの属性を省略しても、アプリケーションのOpenAPI（とドキュメント）で生成されたJSON Schemaが完全なモデルになるからです。

		```Python hl_lines="29 35"

Copy link

Contributor

tokusumiAug 21, 2020

Choose a reason for hiding this comment

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

hl_lines="31 37"

docs/ja/docs/tutorial/response-model.md Outdated


		もし`set`を使用することを忘れて、代わりに`list`や`tuple`を使用しても、FastAPIはそれを`set`に変換して正しく動作します:

		```Python hl_lines="29 35"

Copy link

Contributor

tokusumiAug 21, 2020

Choose a reason for hiding this comment

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

hl_lines="31 37"

docs/ja/docs/tutorial/response-model.md Outdated


		## まとめ

		path operationデコレータの`response_model`パラメータを使用して、レスポンスモデルを定義し、特にプライベートデータがフィルタリングされていることを確認します。

Copy link

Contributor

tokusumiAug 21, 2020

Choose a reason for hiding this comment

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

path operationデコレータのresponse_modelパラメータを使用して、レスポンスモデルを定義し、特にプライベートデータがフィルタリングされていることを確認します。

update

d99da75

Copy link

ContributorAuthor

SwftAlpc commentedAug 22, 2020

@tokusumi
Thanks for your review!
I fixed it.

Copy link

Contributor

github-actionsbot commentedAug 22, 2020

📝 Docs preview for commitd99da75 at:https://5f4160a0442f8ee17ac9c500--fastapi.netlify.app

tokusumi approved these changes

Aug 22, 2020

View reviewed changes

tiangolo added awaiting-review lang-all

Translations

lang-jaJapanese translations labels

Aug 29, 2020

SwftAlpc added2 commits

August 31, 2020 16:54

Merge remote-tracking branch 'upstream/master' into master

de3db05

resolve conflict

bd47a16

Copy link

Contributor

github-actionsbot commentedAug 31, 2020

📝 Docs preview for commitbd47a16 at:https://5f4cbfa2b85066092143d2cc--fastapi.netlify.app

Attsun1031 reviewed

Sep 30, 2020

View reviewed changes

Copy link

Contributor

Attsun1031 left a comment•
edited
Loading

Choose a reason for hiding this comment

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

Thank you for your work 👍
Please check out some of the comments I made.

docs/ja/docs/tutorial/response-model.md OutdatedShow resolvedHide resolved

docs/ja/docs/tutorial/response-model.mdShow resolvedHide resolved

docs/ja/docs/tutorial/response-model.md OutdatedShow resolvedHide resolved

Update docs/ja/docs/tutorial/response-model.md

2e849a1

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

SwftAlpcand others added10 commits

October 29, 2020 20:16

Update docs/ja/docs/tutorial/response-model.md

b6d8db3

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

Update docs/ja/docs/tutorial/response-model.md

45195db

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

Update docs/ja/docs/tutorial/response-model.md

3694f27

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

Update docs/ja/docs/tutorial/response-model.md

044cd78

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

Update docs/ja/docs/tutorial/response-model.md

4f1c2f8

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

Update docs/ja/docs/tutorial/response-model.md

858e501

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

Update docs/ja/docs/tutorial/response-model.md

fed3a74

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

Update docs/ja/docs/tutorial/response-model.md

7086a33

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

Update docs/ja/docs/tutorial/response-model.md

88736be

Co-authored-by: atsumi <atsumi.tatsuya@gmail.com>

resolve conflict

f82f73d

Copy link

Contributor

github-actionsbot commentedOct 29, 2020

📝 Docs preview for commitf82f73d at:https://5f9aa66b8b8d8319b2fce839--fastapi.netlify.app

delhi09 reviewed

Dec 22, 2020

View reviewed changes

Copy link

delhi09 left a comment

Choose a reason for hiding this comment

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

It looks mostly good.
I pointed out some details.

docs/ja/docs/tutorial/response-model.md

		* 出力データをモデルのデータに限定します。これがどのように重要なのか以下で見ていきましょう。

		!!! note "技術詳細"
		レスポンスモデルは、関数の戻り値のアノテーションではなく、このパラメータで宣言されています。なぜなら、パス関数は実際にはそのレスポンスモデルを返すのではなく、`dict`やデータベースオブジェクト、あるいは他のモデルを返し、`response_model`を使用してフィールドの制限やシリアライズを行うからです。

Copy link

delhi09Dec 22, 2020

Choose a reason for hiding this comment

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

It may be a little better to modify "このパラメータ" to "関数のパラメータ" because it's a little hard for readers to undeastand what “この” refers to.

docs/ja/docs/tutorial/response-model.md

		}
		```

		FastAPIは十分に賢いので（実際には、Pydanticが十分に賢い）`description`や`tax`、`tags`はデフォルト値と同じ値を持っているにもかかわらず、明示的に設定されていることを理解しています。（デフォルトから取得するのではなく）

Copy link

delhi09Dec 22, 2020

Choose a reason for hiding this comment

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

It's better to modify "デフォルトから取得するのではなく" to "デフォルトから取得されたのではなく" because " taken from" is the passive voice.

docs/ja/docs/tutorial/response-model.md


		### `response_model_include`と`response_model_exclude`

		path operationデコレータとして`response_model_include`と`response_model_exclude`も使用することができます。

Copy link

delhi09Dec 22, 2020

Choose a reason for hiding this comment

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

It's a little better to add "のパラメータ" after "path operationデコレータ".

docs/ja/docs/tutorial/response-model.md


		path operationデコレータとして`response_model_include`と`response_model_exclude`も使用することができます。

		属性名を持つ`str`の`set`を受け取り、含める(残りを省略する)か、除外(残りを含む)します。

Copy link

delhi09Dec 22, 2020

Choose a reason for hiding this comment

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

I think the following sentence is better .
「これらのパラメータは、含める(残りを省略する)もしくは除外(残りを含む)する対象の属性名のstrのsetを受け取ります。」

[理由]

ベースの文は"They take a set of str"です。
"str"に"with the name of the attributes"がかかっています。
"attributes"に"to include (omitting the rest) or to exclude (including the rest)."がかかっています。("attributes"は"include","exclude"の目的語です。)