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(core): fix Simplified Chinese README #3541

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

Conversation

LearningPawn
Copy link

@LearningPawn LearningPawn commented Jul 4, 2023

  1. Corrected some wrong simplified Chinese characters like 時 -> 时, 裝 -> 装;
  2. Corrected some wrong meaning, like 近 10 年 means In recent 10 years, but actually it is 10 years of in the source English README.md and the demo's dataset Trips data is from 2009 to 2019, 1.6 亿 means 160 million not 1.6 billion;
  3. Corrected some wording that was not clear and coherent, and Removed single line breaks. A single line break will not generate a break but a space by Github markdown renderer, and make a whole chinese word split, just like 阅读 -> 阅 读, Reading -> Re ading;
  4. Add official docs links about InfluxDB Line Protocol & PostgreSQL Wire Protocol;
  5. Translate img tags' descriptions after alt=;
  6. Change wrong html anchor link href='#contribute' -> href='#贡献' it works in my preview;
  7. Update wrong png url from /.github/tsbs-results.png to /.github/readme-benchmark.png.

btw, during the editing README.zh-cn.md, I was confused about the sentence "...to correlate data from multiple sources using relational and time-series joins" in the source English README.md, Do you mean "...to correlate data from multiple sources which are using relational and time-series joins"?
Confusing sentence

Corrected some wrong simplified Chinese characters;
Corrected some wording that was not clear and coherent;
Add official docs links about `Line Protocol` & `Wire Protocol`;
Translate img tags' descriptions under `alt=` ;
Update wrong png url.
@CLAassistant
Copy link

CLAassistant commented Jul 4, 2023

CLA assistant check
All committers have signed the CLA.

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

2 participants