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(zh-cn): update translations & improve readability #39

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

Hephaest
Copy link
Contributor

@Hephaest Hephaest commented Jan 18, 2025

Hi @zakirullin,

I saved your blog a few weeks ago through my RSS reader but only got a chance to take a look this week. 😅 During the reading, I found you provided a Chinese version under the repo. I was surprised, as well as excited.

But when I went through the Chinese README, I found something went wrong:

  • ⚠️ Outdated Content. Especially Layered Architecture section.
  • 😐 Formal/Serious Tone. I believe this blog could benefit anyone eager to learn so I soften the tone and make it look more interesting and friendly to readers.
  • 🧠 Extraneous Terms. I tried to eliminate the unnecessary terms which are not mentioned in the original blog, as it will help reduce readers' cognitive loads.
  • 🐛 Wrong Translations. Some words/sentences were translated completely wrong and readers might get confused.
  • 🧩 Inconsistent formats. I use more standard ways for denotation symbols (e.g., ——, , ). In addition, I help bridge the context gap so readers can easily understand and grasp the ideas.

I have cross-validated Chinese translations with the original blog, and improved readability as much as possible. Please let me know if there is anything I can assist you with! 😃

@zakirullin
Copy link
Owner

@Hephaest Hi!

Just wow 😱! You did such a great job fixing it all. Indeed, Chinese translation lagged behind after we made several updates to the article.

I tried to eliminate the unnecessary terms which are not mentioned in the original blog, as it will help reduce readers' cognitive loads.

This is especially impressive! That's exactly the same thing I was trying to achieve when writing the article. No extra terms that take up our mental capacity.

Thank you so much for your work, I am very happy to see all this happening :)

@zakirullin zakirullin merged commit 36fef64 into zakirullin:main Jan 20, 2025
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