Thanks to visit codestin.com
Credit goes to github.com

Skip to content

🌐 Add Japanese translation for docs/ja/docs/tutorial/dependencies/index.md#1957

Closed
SwftAlpc wants to merge 24 commits into
fastapi:masterfrom
SwftAlpc:translation/ja-tutorial-dependencies-md
Closed

🌐 Add Japanese translation for docs/ja/docs/tutorial/dependencies/index.md#1957
SwftAlpc wants to merge 24 commits into
fastapi:masterfrom
SwftAlpc:translation/ja-tutorial-dependencies-md

Conversation

@SwftAlpc

@SwftAlpc SwftAlpc commented Aug 23, 2020

Copy link
Copy Markdown
Contributor

This PR translates the tutorial/dependencies/index.md to Japanese.
Please review tutorial/dependencies/index.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.

@SwftAlpc SwftAlpc mentioned this pull request Aug 23, 2020
@codecov

codecov Bot commented Aug 23, 2020

Copy link
Copy Markdown

Codecov Report

Patch and project coverage have no change.

Comparison is base (cf73051) 100.00% compared to head (ae46000) 100.00%.

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

Additional details and impacted files
@@             Coverage Diff             @@
##            master     #1957     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files          540       239    -301     
  Lines        13969      7079   -6890     
===========================================
- Hits         13969      7079   -6890     

see 356 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit e03092e at: https://5f426abdc86c106786dfb101--fastapi.netlify.app

@tokusumi tokusumi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thank you for translating difficult chapter!

Comment thread docs/ja/docs/tutorial/dependencies/index.md Outdated
Comment thread docs/ja/docs/tutorial/dependencies/index.md Outdated
Comment thread docs/ja/docs/tutorial/dependencies/index.md Outdated
Comment thread docs/ja/docs/tutorial/dependencies/index.md Outdated
Comment thread docs/ja/docs/tutorial/dependencies/index.md Outdated
@SwftAlpc

Copy link
Copy Markdown
Contributor Author

@tokusumi
I approved your suggetions! Thank you!

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit ae46000 at: https://5f9ae6451c44a80b3723e443--fastapi.netlify.app

@tokusumi tokusumi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 🚀

@sh0nk sh0nk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let me add just a couple of suggestions against typo and nitpicks!


## 「依存性注入」とは

**「依存性注入」** とは、プログラミングにおいて、コード(この場合は、*path operation関数*)が動作したり使用したりするために必要なもの(「依存関係」)を宣言する方法があることを意味します:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
**「依存性注入」** とは、プログラミングにおいて、コード(この場合は、*path operation関数*)が動作したり使用したりするために必要なもの(「依存関係」)を宣言する方法があることを意味します:
**「依存性注入」** とは、プログラミングにおいて、コード(この場合は、*path operation関数*)が動作したり使用したりするために必要なもの(「依存関係」)を宣言する方法があることを意味します


しかし、この方法では **依存性注入** システムがどのように機能するかに焦点を当てることができます。

### 依存関係の作成

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
### 依存関係の作成
### 依存関係 ("dependable") の作成

Add an implication of , or "dependable", which will be appeared on the following parts as well.


## `async`にするかどうか

依存関係は **FastAPI**(*path operation関数*と同じ)からも呼び出されるため、関数を定義する際にも同じルールが適用されます。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
依存関係は **FastAPI***path operation関数*と同じ)からも呼び出されるため、関数を定義する際にも同じルールが適用されます。
依存関係も*path operation関数*と同様に) **FastAPI** から呼び出されるため、関数を定義する際にも同じルールが適用されます。

To clarify the meaning of the sentence.


## **FastAPI** プラグイン

統合や「プラグイン」は **依存性注入** システムを使って構築することができます。しかし、実際には、**「プラグイン」を作成する必要はありません**。依存関係を使用することで、無限の数の統合やインタラクションを宣言することができ、それが**path operation関数*で利用可能になるからです。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
統合や「プラグイン」は **依存性注入** システムを使って構築することができます。しかし、実際には、**「プラグイン」を作成する必要はありません**。依存関係を使用することで、無限の数の統合やインタラクションを宣言することができ、それが**path operation関数*で利用可能になるからです。
統合や「プラグイン」は **依存性注入** システムを使って構築することができます。しかし、実際には、**「プラグイン」を作成する必要はありません**。依存関係を使用することで、無限の数の統合やインタラクションを宣言することができ、それが*path operation関数*で利用可能になるからです。

typo


## OpenAPIとの統合

依存関係(およびサブ依存関係)のすべてのリクエスト宣言、検証、および要件は、同じOpenAPIスキーマに統合されます。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
依存関係(およびサブ依存関係)のすべてのリクエスト宣言、検証、および要件は、同じOpenAPIスキーマに統合されます。
依存関係(およびサブ依存関係)のすべてのリクエスト宣言、バリデーション、および要件は、同じOpenAPIスキーマに統合されます。

To be consistent with #L207. (The other way round is fine too.)

@tiangolo tiangolo changed the title Add tutorial/dependencies/index.md Japanese ver 🌐 Add Japanese translation for docs/ja/docs/tutorial/dependencies/index.md Jun 26, 2023
@tiangolo

Copy link
Copy Markdown
Member

📝 Docs preview for commit fcb10ad at: https://6499cb4e7bcac500837f92f0--fastapi.netlify.app

@tiangolo

Copy link
Copy Markdown
Member

As this was handled in #1958, I'll go ahead and close this one. Thank you! ☕

@tiangolo tiangolo closed this Jan 15, 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-ja Japanese translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants