Skip to content

spec: fix some typo in chinese translation#1068

Merged
duglin merged 2 commits into
cloudevents:mainfrom
boliza:spec/fix_typo
Sep 15, 2022
Merged

spec: fix some typo in chinese translation#1068
duglin merged 2 commits into
cloudevents:mainfrom
boliza:spec/fix_typo

Conversation

@boliza
Copy link
Copy Markdown
Contributor

@boliza boliza commented Aug 30, 2022

Signed-off-by: Ranger Tsao cao.zhifu@gmail.com

fix some typo and one broken link

Signed-off-by: Ranger Tsao <cao.zhifu@gmail.com>
- 示例
- com.github.pull.create
- com.example.object.delete.v2
- com.github.pull_request.opened
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

seems like we should make this change in the English version too for consistency... if we want to change it

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@duglin I checked the English version for these two lines, they've been changed already.

spec/cloudevents/spec.md

Lines 350 to 351 in 676cef9

- com.github.pull_request.opened
- com.example.object.deleted.v2

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

cool thanks!

- com.github.pull.create
- com.example.object.delete.v2
- com.github.pull_request.opened
- com.example.object.deleted.v2
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

here too

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ditto the comments above

@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Aug 30, 2022

@JieDing can you review? thanks

@JieDing
Copy link
Copy Markdown
Member

JieDing commented Aug 30, 2022

@JieDing can you review? thanks

Sure, I'll review it.

Copy link
Copy Markdown
Member

@JieDing JieDing left a comment

Choose a reason for hiding this comment

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

Thanks a ton for your careful check for the translation. Here are some suggestions.

Comment thread cloudevents/languages/zh-CN/spec.md Outdated
- 描述: `data` 值的内容类型。 此属性使 `data` 能够承载任何类型的内容,
因此格式和编码可能与所选事件格式的不同。
例如,使用 [JSON envelope](../../formats/json-format.md#3-envelope)格式呈现的事件可能在数据中携带 XML 的payload,这个属性可以用来通知消费者
例如,使用 [JSON envelope](../../formats/json-format.md#3-envelope)格式呈现的事件可能在数据中携带 XML 的 payload,这个属性可以用来通知消费者
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
例如,使用 [JSON envelope](../../formats/json-format.md#3-envelope)格式呈现的事件可能在数据中携带 XML 的 payload,这个属性可以用来通知消费者
例如,使用 [JSON envelope](../../formats/json-format.md#3-envelope)格式呈现的事件可能在数据中携带 XML 的有效负载,这个属性可以用来通知消费者

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

OK.

@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Sep 7, 2022

@boliza did you want to make any of the suggested updates?

@boliza
Copy link
Copy Markdown
Contributor Author

boliza commented Sep 9, 2022

sorry for a bit busy these days. will update this pr doday

Signed-off-by: Ranger Tsao <cao.zhifu@gmail.com>
@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Sep 9, 2022

@JieDing are you ok with the PR now?

@JieDing
Copy link
Copy Markdown
Member

JieDing commented Sep 9, 2022

@JieDing are you ok with the PR now?

@duglin LGTM

@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Sep 15, 2022

Approved on the 9/15 call

@duglin duglin merged commit 019a339 into cloudevents:main Sep 15, 2022
sasha-tkachev pushed a commit to sasha-tkachev/cloudevents-spec that referenced this pull request Oct 14, 2022
* spec: fix some typo in chinese translation

Signed-off-by: Ranger Tsao <cao.zhifu@gmail.com>

* spec: apply review comment

Signed-off-by: Ranger Tsao <cao.zhifu@gmail.com>

Signed-off-by: Ranger Tsao <cao.zhifu@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants