Skip to content

Conversation

@yanrongzhen
Copy link
Contributor

Fixes #4663 .

Modifications

Add documentation for wecom sink connector.

Comment on lines +6 to +8
2. 启用 sinkConnector 并检查 `sink-config.yml`。
3. 向 eventmesh 发送带有在 `pubSubConfig.subject` 中定义的主题消息。
```yaml
Copy link
Member

Choose a reason for hiding this comment

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

The translation of the third point is not quite smooth. Can it be "使用在pubSubConfig.subject中指定的Topic,向EventMesh发送消息。"?

Copy link
Member

Choose a reason for hiding this comment

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

Fixed in #4668.

@xwm1992 xwm1992 added this to the 1.10 milestone Dec 16, 2023
@codecov
Copy link

codecov bot commented Dec 16, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f17b82e) 17.39% compared to head (9eddc1d) 17.39%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #4664   +/-   ##
=========================================
  Coverage     17.39%   17.39%           
  Complexity     1758     1758           
=========================================
  Files           797      797           
  Lines         29787    29787           
  Branches       2575     2575           
=========================================
  Hits           5180     5180           
  Misses        24129    24129           
  Partials        478      478           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

3. send a message to eventmesh with the topic defined in `pubSubConfig.subject`
```yaml
pubSubConfig:
meshAddress: 127.0.0.1:10000
Copy link
Member

Choose a reason for hiding this comment

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

Could you change it to the following format like this document? This may be clearer.

能否像该文档一样改成如下这样?这样也许更明确。

# default port is 10000
 meshAddress: your.eventmesh.server:10000 

Copy link
Member

Choose a reason for hiding this comment

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

Fixed in #4668.

@@ -0,0 +1,22 @@
# eventmesh-connector-wecom
Copy link
Member

Choose a reason for hiding this comment

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

This connector has requirements for data properties, but it seems that your document does not specify them. #4591 (comment)

该Connector对数据的属性是有要求的,您的文档好像没有说明。#4591 (comment)

Copy link
Member

Choose a reason for hiding this comment

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

Fixed in #4668.

@mxsm mxsm merged commit 8d5b39b into apache:master Dec 16, 2023
@@ -0,0 +1,22 @@
# eventmesh-connector-slack

## SlackSinkConnector:从 eventmesh 到 slack。
Copy link
Member

@pandaapo pandaapo Dec 16, 2023

Choose a reason for hiding this comment

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

Please correct these two incorrect titles in the future. This is wecom, not slack.

Additionally, the documents (dingtalk, wecom) you wrote did not inform users as previously mentioned which attribute values are valid. It will be appreciate if you can supplement them in the future.

请在后期修改一下这两个错误的标题,这是wecom,不是slack。
另外,您写的文档(钉钉企业微信)并没有像之前说过的那样会告知用户哪些属性值是合法的,希望您后期能够进行补充。

Copy link
Member

Choose a reason for hiding this comment

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

Fixed in #4668.

Pil0tXia added a commit to Pil0tXia/eventmesh that referenced this pull request Dec 16, 2023
pandaapo pushed a commit that referenced this pull request Dec 17, 2023
* Unify 'LiteMessage' to 'EventMeshMessage'

* Sync connectors doc

* Format RabbitMQ & 企业微信

* Format Slack & 钉钉

* Format HTTP & MongoDB

* Unify 'EventMesh Runtime'

* Unify 'EventMesh'

* Fix translation #4664 (comment)

* Explain additional attributes #4591 (comment)

* Append: Unify remaining 'EventMesh'

* Beautify meshAddress

* Add and correct attributes
xuhongjia pushed a commit to Deckers-Ohana/eventmesh that referenced this pull request Mar 13, 2025
xuhongjia pushed a commit to Deckers-Ohana/eventmesh that referenced this pull request Mar 13, 2025
…ache#4668)

* Unify 'LiteMessage' to 'EventMeshMessage'

* Sync connectors doc

* Format RabbitMQ & 企业微信

* Format Slack & 钉钉

* Format HTTP & MongoDB

* Unify 'EventMesh Runtime'

* Unify 'EventMesh'

* Fix translation apache#4664 (comment)

* Explain additional attributes apache#4591 (comment)

* Append: Unify remaining 'EventMesh'

* Beautify meshAddress

* Add and correct attributes
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.

[Doc] Documentation Needed for eventmesh-connector-wecom.

5 participants