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

docs(cn): translate docs/faq-ajax.md into Chinese #157

Merged
merged 12 commits into from Apr 1, 2019

Conversation

hateonion
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Mar 4, 2019

Deploy preview for cn-reactjs ready!

Built with commit 196f15c

https://deploy-preview-157--cn-reactjs.netlify.com

@netlify
Copy link

netlify bot commented Mar 4, 2019

Deploy preview for zh-hans-reactjs ready!

Built with commit 196f15c

https://deploy-preview-157--zh-hans-reactjs.netlify.com

@QC-L QC-L added the Pending Review 已翻译,待校对阶段 label Mar 5, 2019
@QC-L QC-L mentioned this pull request Mar 5, 2019
Copy link
Collaborator

@LeoEatle LeoEatle left a comment

Choose a reason for hiding this comment

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

两处建议

content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
LeoEatle and others added 2 commits March 5, 2019 22:31
Co-Authored-By: hateonion <ouyang_yingjie@outlook.com>
Co-Authored-By: hateonion <ouyang_yingjie@outlook.com>
LeoEatle
LeoEatle previously approved these changes Mar 6, 2019
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
@yuqingc
Copy link
Member

yuqingc commented Mar 8, 2019

单词两边需要空格

Copy link
Member

@yuqingc yuqingc left a comment

Choose a reason for hiding this comment

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

下文都统一改一下

@@ -6,19 +6,19 @@ layout: docs
category: FAQ
---

### How can I make an AJAX call? {#how-can-i-make-an-ajax-call}
### 我怎样在React中发起AJAX请求{#how-can-i-make-an-ajax-call}
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
### 我怎样在React中发起AJAX请求{#how-can-i-make-an-ajax-call}
### 我怎样在 React 中发起 AJAX 请求{#how-can-i-make-an-ajax-call}

@hateonion
Copy link
Contributor Author

下文都统一改一下

全部都已经更正啦

Copy link
Member

@yuqingc yuqingc left a comment

Choose a reason for hiding this comment

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

使用中文标点,下文请统一修改。


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).
在React开发中,你能使用任何你喜欢的 AJAX 库,比如社区比较流行的[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
在React开发中,你能使用任何你喜欢的 AJAX 库,比如社区比较流行的[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)
在React开发中,你能使用任何你喜欢的 AJAX 库,比如社区比较流行的[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.

辛苦了

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已经修复

Copy link
Member

@yuqingc yuqingc left a comment

Choose a reason for hiding this comment

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

这种超链接的内容如果是英文的话,也需要两边有空格


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).
在React开发中,你能使用任何你喜欢的 AJAX 库,比如社区比较流行的[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
在React开发中,你能使用任何你喜欢的 AJAX 库,比如社区比较流行的[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)
在React开发中,你能使用任何你喜欢的 AJAX 库,比如社区比较流行的 [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
Contributor Author

Choose a reason for hiding this comment

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

已经更正,如果还有任何lint问题,请一下全部提出哈,这样我也方便一次全部进行修复

Copy link
Member

Choose a reason for hiding this comment

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

好的,不好意思啊,我之前没有看出来。。🤣

Copy link
Contributor Author

Choose a reason for hiding this comment

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

哈哈哈 没事没事,没有blame你哈。 非常感谢你这么仔细的review☺️

content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
yuqingc and others added 3 commits March 15, 2019 13:10
Co-Authored-By: hateonion <ouyang_yingjie@outlook.com>
Co-Authored-By: hateonion <ouyang_yingjie@outlook.com>
@hateonion
Copy link
Contributor Author

已经全部更正

@LeoEatle LeoEatle added Pending Re-Review 已修改,待审校阶段 and removed Pending Review 已翻译,待校对阶段 labels Mar 29, 2019
LeoEatle
LeoEatle previously approved these changes Mar 29, 2019
Copy link
Member

@yuqingc yuqingc left a comment

Choose a reason for hiding this comment

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

😂 还有两处格式问题你遗漏了,上次没有改全,再麻烦改一下,然后全文统一找一下有什么格式问题。

content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
Co-Authored-By: hateonion <ouyang_yingjie@outlook.com>
@hateonion
Copy link
Contributor Author

已经更正 @yuqingc

yuqingc
yuqingc previously approved these changes Mar 31, 2019
Copy link
Member

@QC-L QC-L left a comment

Choose a reason for hiding this comment

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

基本上没什么问题,简单修改下即可

content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
Co-Authored-By: hateonion <ouyang_yingjie@outlook.com>
@hateonion
Copy link
Contributor Author

Fixed @QC-L

Copy link
Member

@QC-L QC-L left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@yuqingc yuqingc left a comment

Choose a reason for hiding this comment

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

lgtm

@QC-L QC-L merged commit 5f9cd32 into reactjs:master Apr 1, 2019
OhIAmFine pushed a commit to OhIAmFine/zh-hans.reactjs.org that referenced this pull request May 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pending Re-Review 已修改,待审校阶段
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants