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

Translate faq-ajax #10

Merged
merged 3 commits into from Feb 2, 2019
Merged

Translate faq-ajax #10

merged 3 commits into from Feb 2, 2019

Conversation

ossan-engineer
Copy link
Contributor

This PR is part of Issue #4.

@smikitky smikitky self-requested a review February 1, 2019 11:32

You can use any AJAX library you like with React. Some popular ones are [Axios](https://github.com/axios/axios), [jQuery AJAX](https://api.jquery.com/jQuery.ajax/), and the browser built-in [window.fetch](https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API).
任意のAJAXライブラリをReactと共に利用可能です。人気のあるものとしては、[Axios](https://github.com/axios/axios)[jQuery AJAX](https://api.jquery.com/jQuery.ajax/)、ブラウザ組み込みの [window.fetch](https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API)などがあります。

Choose a reason for hiding this comment

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

MDNのリンクは https://developer.mozilla.org/ja/docs/Web/API/Fetch_API にしても良さそう

Copy link
Member

Choose a reason for hiding this comment

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

MDNのリンクはドキュメント中に数十カ所あるので、日本語版にはまとめて置換するほうが効率的かなということで、一応そのままにするという形でお願いしています。

@tesseralis tesseralis mentioned this pull request Feb 1, 2019
90 tasks
Copy link
Member

@smikitky smikitky left a comment

Choose a reason for hiding this comment

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

大部分はスペースの置き方などの些細な問題なのですがご確認の上対応をお願いします。

@@ -1,24 +1,24 @@
---
id: faq-ajax
title: AJAX and APIs
title: AJAXとAPI
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
title: AJAXとAPI
title: AJAX と API

英字と和文との間に半角スペースをお願いします。 TRANSLATION.md をご一読ください。

permalink: docs/faq-ajax.html
layout: docs
category: FAQ
---

### How can I make an AJAX call?
### AJAXコールするには?
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
### AJAXコールするには
### AJAX コールをする方法は


You can use any AJAX library you like with React. Some popular ones are [Axios](https://github.com/axios/axios), [jQuery AJAX](https://api.jquery.com/jQuery.ajax/), and the browser built-in [window.fetch](https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API).
任意のAJAXライブラリをReactと共に利用可能です。人気のあるものとしては、[Axios](https://github.com/axios/axios)[jQuery AJAX](https://api.jquery.com/jQuery.ajax/)、ブラウザ組み込みの [window.fetch](https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API)などがあります。
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
任意のAJAXライブラリをReactと共に利用可能です。人気のあるものとしては、[Axios](https://github.com/axios/axios)[jQuery AJAX](https://api.jquery.com/jQuery.ajax/)、ブラウザ組み込みの [window.fetch](https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API)などがあります。
任意の AJAX ライブラリを React と共に利用可能です。人気のあるものとしては、[Axios](https://github.com/axios/axios)[jQuery AJAX](https://api.jquery.com/jQuery.ajax/)、ブラウザ組み込みの [window.fetch](https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API) などがあります。


### Where in the component lifecycle should I make an AJAX call?
### コンポーネントのどのライフサイクルでAJAXコールすべきか?
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
### コンポーネントのどのライフサイクルでAJAXコールすべきか
### コンポーネントのどのライフサイクルで AJAX コールすべきか


You should populate data with AJAX calls in the [`componentDidMount`](/docs/react-component.html#mounting) lifecycle method. This is so you can use `setState` to update your component when the data is retrieved.
AJAXコールによるデータ取得は[`componentDidMount`](/docs/react-component.html#mounting)で行うと良いでしょう。データ取得後に `setState` でコンポーネントを更新できるようにするためです。
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
AJAXコールによるデータ取得は[`componentDidMount`](/docs/react-component.html#mounting)で行うと良いでしょう。データ取得後に `setState` でコンポーネントを更新できるようにするためです。
AJAX コールによるデータ取得は [`componentDidMount`](/docs/react-component.html#mounting) のライフサイクルメソッドで行うべきです。データ取得後に `setState` でコンポーネントを更新できるようにするためです。

この should は React の思想的に比較的強い should だと思われるので強めにしました


### Example: Using AJAX results to set local state
### 例:AJAXの通信結果をローカルstateで利用する
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
### 例:AJAXの通信結果をローカルstateで利用する
### 例:ローカル state に AJAX の通信結果をセットする


The component below demonstrates how to make an AJAX call in `componentDidMount` to populate local component state.
下記のコンポーネントは、 `componentDidMount` でAJAXコールして得られたデータをローカルコンポーネントのstateに流し込んでいます。
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
下記のコンポーネントは、 `componentDidMount` でAJAXコールして得られたデータをローカルコンポーネントのstateに流し込んでいます
下記のコンポーネントは、`componentDidMount` で AJAX コールして得られたデータをローカルコンポーネントの state に流し込んでいます


The example API returns a JSON object like this:
サンプルAPIが返すJSONオブジェクトはこのようになります:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
サンプルAPIが返すJSONオブジェクトはこのようになります
このサンプル API が返す JSON オブジェクトはこのようなものです

@smikitky
Copy link
Member

smikitky commented Feb 1, 2019

ありがとうございます!

@ossan-engineer
Copy link
Contributor Author

@smikitky レビューありがとうございます!諸々修正しましたのでご確認お願いいたします:pray:

@smikitky smikitky merged commit ce556dd into reactjs:master Feb 2, 2019
@smikitky
Copy link
Member

smikitky commented Feb 2, 2019

ありがとうございました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants