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

🌐 Add Chinese translation for Advanced - Path Operation Advanced Configuration #1447

Conversation

RunningIkkyu
Copy link
Contributor

Add Chinese docs for advanced/path-operation-advanced-configuration.md. Could you help review plz. @waynerv @Dustyposa

@codecov
Copy link

codecov bot commented May 21, 2020

Codecov Report

Merging #1447 (4644e67) into master (b8331b1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1447   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          240       240           
  Lines         7101      7101           
=========================================
  Hits          7101      7101           

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 8997f96...4644e67. Read the comment docs.

!!! warning
如果你并非 OpenAPI 的「专家」,你可能不需要这部分内容。

你可以通过参数 `operation_id` 将 OpenAPI 的 `operationId` 设置为在路径操作中使用。
Copy link
Contributor

Choose a reason for hiding this comment

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

你可以在路径操作中通过参数 operation_id 设置要使用的 OpenAPI operationId

Copy link
Contributor

Choose a reason for hiding this comment

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

The segmentation should be "You can set the OpenAPI operationId to be used | in your path operation | with the parameter operation_id."


你可以通过参数 `operation_id` 将 OpenAPI 的 `operationId` 设置为在路径操作中使用。

务必确保每次操作的 `operation_id` 都是唯一的。
Copy link
Contributor

Choose a reason for hiding this comment

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

"each operation" could translate to "每个路径操作“


如果你想用你的 API 的函数名作为 `operationId` 的名字,你可以遍历一遍 API 的函数名,然后使用他们的 `APIRoute.name` 重写每个 *路径操作* 的 `operation_id`。

在此之前你要给所有函数添加 *路径操作*。
Copy link
Contributor

Choose a reason for hiding this comment

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

你应该在添加了所有路径操作之后执行此操作。

```

!!! tip
如果你手动调动 `app.openapi()`,你应该在此之前更新 `operationId`。
Copy link
Contributor

Choose a reason for hiding this comment

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

”call“ always translate to "调用".Is here a typo?

@RunningIkkyu
Copy link
Contributor Author

@waynerv Thanks for your review.

@tiangolo tiangolo added lang-all Translations lang-zh Chinese translations labels Jun 7, 2020
@RunningIkkyu
Copy link
Contributor Author

@Dustyposa @phy25 @yanever @Laineyzhang55 Please take a look.

@tiangolo tiangolo changed the title Add Chinese docs for advanced/path-operation-advanced-configuration.md 🌐 Add Chinese translation for Advanced - Path Operation Advanced Configuration Nov 25, 2020
@tiangolo
Copy link
Owner

Thank you @RunningIkkyu ! 👏 👏 🍰

And thanks @waynerv and @Laineyzhang55 for the reviews 🙇‍♂️ ☕

@tiangolo tiangolo merged commit 410da16 into tiangolo:master Nov 25, 2020
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.

None yet

4 participants