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 minio-js docs to Chinese. #650

Merged
merged 3 commits into from
Dec 19, 2017
Merged

translate minio-js docs to Chinese. #650

merged 3 commits into from
Dec 19, 2017

Conversation

dingjs
Copy link
Contributor

@dingjs dingjs commented Nov 20, 2017

translate minio-js docs to Chinese.

@deekoder deekoder self-requested a review November 20, 2017 23:41
@harshavardhana harshavardhana requested review from harshavardhana and yihuaf and removed request for deekoder November 29, 2017 01:22
@harshavardhana
Copy link
Member

Moved review from @deekoder due to unavailability.

Copy link

@yihuaf yihuaf left a comment

Choose a reason for hiding this comment

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

Sorry for the lateness. not sure how this slipped through the crack.

@dingjs Not sure if you want to translate the comments in the code to chinese as well.

README_zh_CN.md Outdated
npm install --save minio
```

## 下载源码安装
Copy link

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.

Acturally I'm not.The code explains itself.Maybe I should change the comment back to English,for consistency.

Copy link

Choose a reason for hiding this comment

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

@dingjs SGTM, as long as it's consistent.

@deekoder May be you can comment on if Minio as a whole project would like comments in the code to be translated or remain in English? This can be set as president for future translation work.

Copy link
Contributor

Choose a reason for hiding this comment

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

comments in code can be in english


__示例__

单个对象的最大大小限制在5TB。putObject在对象大于5MiB时,自动使用multiple parts方式上传。这就当上传失败时,客户端只需要上传未成功的部分即可(类似断点上传)。上传的对象使用MD5SUM签名进行完整性验证。
Copy link

Choose a reason for hiding this comment

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

这就当上传失败时->这样的话,当上传失败的时

__示例__


单个对象的最大大小限制在5TB。fPutObject在对象大于5MiB时,自动使用multiple parts方式上传。这就当上传失败时,客户端只需要上传未成功的部分即可(类似断点上传)。上传的对象使用MD5SUM签名进行完整性验证。
Copy link

Choose a reason for hiding this comment

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

ditto

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ditto
@yihuaf The original docs is ditto too,but I agree with you,it's a little redundant.I removed it in Chinese docs.

harshavardhana
harshavardhana previously approved these changes Dec 13, 2017
@krishnasrinivas
Copy link
Contributor

@harshavardhana this can be taken in right?

@deekoder
Copy link
Contributor

@krishnasrinivas No. We need @yihuaf approval since hes is the key reviewer for this PR.

@dingjs
Copy link
Contributor Author

dingjs commented Dec 15, 2017

@yihuaf ping.

Copy link

@yihuaf yihuaf left a comment

Choose a reason for hiding this comment

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

Apologize for the lateness. Was traveling and had limited access.

@deekoder deekoder removed the blocked label Dec 19, 2017
@deekoder deekoder merged commit 01fa7be into minio:master Dec 19, 2017
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.

5 participants