-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
Add cn translation for serveral sections of advanced user guide. #1804
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1804 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 238 235 -3
Lines 7187 6989 -198
==========================================
- Hits 7187 6989 -198
Continue to review full report at Codecov.
|
📝 Docs preview for commit 5145550 at: https://5f20162d2e9533025d3038f0--fastapi.netlify.app |
That's great! 🎉 Could you separate each file in a different PR? If each file is in a different PR, it will be easier for others to review an entire PR and/or add comments to specific sections. |
The embarrassing thing is these 5 articles have already been translated by @RunningIkkyu . you should follow the contribution guide and check PR list to avoid duplicate work. |
It's my fault that I forgot to check. |
Filtering PR list by label |
As these translations have been added in other PRs I'm gonna close this one. Thanks! ☕ |
including:
index.md
path-operation-advanced-configuration.md
additional-status-codes.md
custom-response.md
response-directly.md
And more will be added later.