Thanks to visit codestin.com
Credit goes to github.com

Skip to content

🌐 Update Chinese translations with new source files#9738

Merged
tiangolo merged 3 commits into
fastapi:masterfrom
mahone3297:docs-upgrade-py-code-from-en-to-zh
Jul 27, 2023
Merged

🌐 Update Chinese translations with new source files#9738
tiangolo merged 3 commits into
fastapi:masterfrom
mahone3297:docs-upgrade-py-code-from-en-to-zh

Conversation

@mahone3297

Copy link
Copy Markdown
Contributor

The docs in language zh is out of date.
So the high line is wrong on the python code, that makes the reader confuse.

I copy the py-code from language en to zh.

@tiangolo

Copy link
Copy Markdown
Member

📝 Docs preview for commit 4efcc69 at: https://64977ec294f5d8307aebe55d--fastapi.netlify.app

@tiangolo tiangolo changed the title [docs] upgrade py-code from en to zh 🌐 Update Chinese translations with new source files Jun 27, 2023
@tiangolo tiangolo added lang-all Translations lang-zh Chinese translations awaiting-review labels Jun 27, 2023
Comment thread docs/zh/docs/tutorial/body-fields.md Outdated
Comment thread docs/zh/docs/tutorial/body-fields.md Outdated
Comment thread docs/zh/docs/tutorial/body-multiple-params.md Outdated
Comment thread docs/zh/docs/tutorial/body-multiple-params.md Outdated
Comment thread docs/zh/docs/tutorial/body-multiple-params.md Outdated
Comment thread docs/zh/docs/tutorial/path-params-numeric-validations.md Outdated
Comment thread docs/zh/docs/tutorial/path-params-numeric-validations.md Outdated
=== "Python 3.6+ non-Annotated"

!!! tip
Prefer to use the `Annotated` version if possible.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Prefer to use the `Annotated` version if possible.
尽可能选择使用 `Annotated` 的版本。

=== "Python 3.10+ non-Annotated"

!!! tip
Prefer to use the `Annotated` version if possible.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Prefer to use the `Annotated` version if possible.
尽可能选择使用 `Annotated` 的版本。

=== "Python 3.6+ non-Annotated"

!!! tip
Prefer to use the `Annotated` version if possible.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Prefer to use the `Annotated` version if possible.
尽可能选择使用 `Annotated` 的版本。

@mkdir700 mkdir700 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Some sentences have not been translated.

mahone3297 and others added 2 commits July 7, 2023 11:38
[docs] translate from en to zh

Co-authored-by: mkdir700 <[email protected]>
[docs] upgrade py-code from en to zh

Co-authored-by: mkdir700 <[email protected]>
@tiangolo

tiangolo commented Jul 7, 2023

Copy link
Copy Markdown
Member

📝 Docs preview for commit 66e8d3d at: https://64a78a5bb0058521e7475fd5--fastapi.netlify.app

@mahone3297

Copy link
Copy Markdown
Contributor Author

Is there anybody to approve this PR? Do I need to follow any procedures?
Two native speaker have reviewed it and I have changed the code based on the review.

@tiangolo

Copy link
Copy Markdown
Member

Awesome, thank you @mahone3297! 🚀

And thanks for the reviews @mkdir700, @Slijeff, @wdh99 🙇

@tiangolo tiangolo merged commit 943baa3 into fastapi:master Jul 27, 2023
erosennin pushed a commit to erosennin/fastapi that referenced this pull request Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-2 lang-all Translations lang-zh Chinese translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants