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: intro/thanks.md #133

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

RLukas2
Copy link

@RLukas2 RLukas2 commented Feb 16, 2022

This is my first time making pull request like this. But I'll make sure to update if any problem occurred or any errors.
Also if there're anyone who gonna make changes, please:

  • Read the text before make changes if there's anything is wrong or typos?
  • Anything need to translate furthur? Please contact me.

Please Reviewers take care of the following aspects:

  1. Are there any typos?
  2. Please read the content of the Pull Request as if you were a beginner and talk about your thoughts after reading the text no matter whether you are familiar with the content or not.
  3. Please evaluate the content of the Pull Request if you are familiar with related concepts.
  4. Please try your best to keep an eye on it until it is merged or closed.

This is my first time making pull request like this. But I'll make sure to update if any problem occurred or any errors.
Also if there're anyone who gonna make changes, please:
- Read the text before make changes if there's anything is wrong or typos?
- Anything need to translate furthur? Please contact me.
Copy link
Member

@ksyx ksyx left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution and hope our content helped you on learning. Your work looks great generally, but please also convert punctuation into english ones.

@@ -1,9 +1,9 @@
disqus:

本项目目前接受捐赠,扫描下方二维码可以投食(请务必备注 `‘捐赠’+ 自己的信息` )。
This project currently accepting your donations. Scan the QR code below to donate.(Be sure to note: `‘Donation’ + Your information` or in Chinese: `‘捐赠’+ 自己的信息`)。
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
This project currently accepting your donations. Scan the QR code below to donate.Be sure to note: `‘Donation’ + Your information` or in Chinese: `‘捐赠’+ 自己的信息`
This project currently accepting your donations. Scan the QR code below to donate. (Be sure to remark `‘Donation’ + Your information`)

We have a Chinese site so just English content on this site is okay :)

Copy link
Author

@RLukas2 RLukas2 Feb 16, 2022

Choose a reason for hiding this comment

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

Ah thank you, I might have forgot when translating because the punctuation mark look the same as English in markdown editor.


所有款项将被用于 **OI Wiki** 的域名、服务器、运维等必须支出
大额捐赠将会记录在本页面下方或日后更合适的位置来表示感谢。
All funds will be used for **OI Wiki** domain、server、all operation and maintenance.
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
All funds will be used for **OI Wiki** domainserverall operation and maintenance.
All funds will be used for **OI Wiki** domain, server, all other operation and maintenance costs.

所有款项将被用于 **OI Wiki** 的域名、服务器、运维等必须支出
大额捐赠将会记录在本页面下方或日后更合适的位置来表示感谢。
All funds will be used for **OI Wiki** domain、server、all operation and maintenance.
Donation with large amount will be included at the bottom of this page or in more appropriate place in the future to express gratitute from us.
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
Donation with large amount will be included at the bottom of this page or in more appropriate place in the future to express gratitute from us.
Donation with large amount will be included at the bottom of this page or in more appropriate place in the future to express our gratitude.

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.

2 participants