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 translation for docs/zh/docs/tutorial/request-files.md #4529

Conversation

ASpathfinder
Copy link
Contributor

  1. Fixed the parts of which the meaning is different to the original English version.
  2. Updated with new content that appeared in the English version.

@ASpathfinder ASpathfinder changed the title Update chinese translation for docs/zh/docs/tutorial/request-files.md Update Chinese translation for docs/zh/docs/tutorial/request-files.md Feb 7, 2022
@tiangolo tiangolo changed the title Update Chinese translation for docs/zh/docs/tutorial/request-files.md 🌐 Update Chinese translation for docs/zh/docs/tutorial/request-files.md May 10, 2022
@tiangolo tiangolo added lang-all Translations lang-zh Chinese translations awaiting-review labels May 10, 2022
@github-actions github-actions bot mentioned this pull request May 10, 2022
@codecov
Copy link

codecov bot commented May 10, 2022

Codecov Report

Merging #4529 (4b2c994) into master (1876ebc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #4529   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          532       532           
  Lines        13672     13672           
=========================================
  Hits         13672     13672           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1876ebc...4b2c994. Read the comment docs.

@github-actions
Copy link
Contributor

📝 Docs preview for commit 73e1c6c at: https://6279ded5c506a72977d14a37--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit c88b1a7 at: https://627cc50656caf93b7cf42d9b--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit 4b2c994 at: https://628b3d0563d6b209c5541e05--fastapi.netlify.app

Copy link

@mcthesw mcthesw left a comment

Choose a reason for hiding this comment

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

Good.
The original version "含 UploadFile 的 File 参数" is a bad expression。

@tiangolo
Copy link
Member

tiangolo commented Sep 4, 2022

Great, thank you @ASpathfinder! 🚀

And thanks for the reviews @LindsayZhou and @mcthesw! 🙇

@tiangolo tiangolo merged commit dc15347 into fastapi:master Sep 4, 2022
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