Skip to content

Conversation

@waynerv
Copy link
Contributor

@waynerv waynerv commented Nov 30, 2020

@RunningIkkyu @maoyibo @yanever Please take a look.

@codecov
Copy link

codecov bot commented Nov 30, 2020

Codecov Report

Merging #2442 (6307b8e) into master (e6f8aa8) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2442   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          242       242           
  Lines         7396      7396           
=========================================
  Hits          7396      7396           

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 d6b5bc9...6307b8e. Read the comment docs.

@waynerv waynerv mentioned this pull request Nov 30, 2020
@github-actions
Copy link
Contributor

📝 Docs preview for commit 1120a44 at: https://5fc4983861324473f1f96709--fastapi.netlify.app

Copy link
Contributor

@maoyibo maoyibo left a comment

Choose a reason for hiding this comment

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

Looks good to me

!!! tip
要了解有关每个状态代码以及适用场景的更多信息,请查看 <a href="https://developer.mozilla.org/en-US/docs/Web/HTTP/Status" class="external-link" target="_blank"><abbr title="Mozilla Developer Network">MDN</abbr> 关于 HTTP 状态码的文档</a>。

## 记住名称的捷径
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we translate "Shortcut" into "快捷方式"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think “捷径” is more concisely here in a title, and their meaning is almost the same.

@tiangolo tiangolo added awaiting-review lang-all Translations lang-zh Chinese translations labels Dec 2, 2020
Copy link

@Laineyzhang55 Laineyzhang55 left a comment

Choose a reason for hiding this comment

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

Good translation.

@tiangolo tiangolo changed the title Add Chinese translation for Tutorial - Response Status Code 🌐 Add Chinese translation for Tutorial - Response Status Code Dec 19, 2020
@tiangolo tiangolo merged commit 3f478b7 into fastapi:master Dec 19, 2020
@tiangolo
Copy link
Member

Awesome! Thank you @waynerv ! 🚀 🙇

And thanks @maoyibo , @yanever , and @Laineyzhang55 for the reviews! ☕ 🍰

@waynerv waynerv deleted the add-chinese-translation-for-reponse-status-code.md branch January 13, 2021 03:03
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