8000 🌐 Update Chinese translations with new source files by mahone3297 · Pull Request #9738 · fastapi/fastapi · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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

🌐 Update Chinese translations with new source files #9738

Merged

Conversation

mahone3297
Copy link
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
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
docs/zh/docs/tutorial/body-fields.md Outdated Show resolved Hide resolved
docs/zh/docs/tutorial/body-fields.md Outdated Show resolved Hide resolved
docs/zh/docs/tutorial/body-multiple-params.md Outdated Show resolved Hide resolved
docs/zh/docs/tutorial/body-multiple-params.md Outdated Show resolved Hide resolved
docs/zh/docs/tutorial/body-multiple-params.md Outdated Show resolved Hide resolved
docs/zh/docs/tutorial/path-params-numeric-validations.md Outdated Show resolved Hide resolved
docs/zh/docs/tutorial/path-params-numeric-validations.md Outdated Show resolved Hide resolved
=== "Python 3.6+ non-Annotated"

!!! tip
Prefer to use the `Annotated` version if possible.
Copy link
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
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
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` 的版本。

Copy link
Contributor
@mkdir700 mkdir700 left a comment

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 <mkdir700@gmail.com>
[docs] upgrade py-code from en to zh

Co-authored-by: mkdir700 <mkdir700@gmail.com>
@tiangolo
Copy link
Member
tiangolo commented Jul 7, 2023

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

@mahone3297
Copy link
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
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.

5 participants
0