Skip to content

Conversation

@Pil0tXia
Copy link
Member

@Pil0tXia Pil0tXia commented Dec 16, 2023

Fixes #4665.

Motivation

#4665

Modifications

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@Pil0tXia Pil0tXia force-pushed the pil0txia_enhance_4665 branch from 18d5c63 to cafa7e3 Compare December 16, 2023 15:59
pubSubConfig:
# default port is 10000
# default port 10000
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.

127.0.0.1:10000 may be optimized to your.eventmesh.server:10000 like this document. The same applies to several other documents.

Copy link
Member Author

@Pil0tXia Pil0tXia Dec 16, 2023

Choose a reason for hiding this comment

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

Updated dingtalk, wocom, slack in ae41084 ~

When using the eventmesh-connector-wecom sinking event, you need to add the corresponding extension filed in CloudEvent:
- When key=`wecomTemplateType`, value=`text`/`markdown`, indicating the text type of the event.
Copy link
Member

Choose a reason for hiding this comment

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

It should be wecomtemplatetype.

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed.

When using the eventmesh-connector-dingtalk sinking event, you need to add the corresponding extension filed in CloudEvent:
- When key=`dingtalkTemplateType`, value=`text`/`markdown`, indicating the text type of the event.
Copy link
Member

Choose a reason for hiding this comment

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

It should be dingtalktemplatetype. Additionally, please help add that when dingtalktemplatetype‘s value is markdown, you can set a title for the markdown text: key=dingtalkmarkdownmessagetitle, value=your title.

应该是dingtalktemplatetype。另外,还请帮忙加上,当dingtalktemplatetype的value是markdown时,可以为markdown文本设置标题:key=dingtalkmarkdownmessagetitle,value=您的标题

Copy link
Member Author

@Pil0tXia Pil0tXia Dec 16, 2023

Choose a reason for hiding this comment

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

Done in 1a2e26a.

@Pil0tXia Pil0tXia requested a review from pandaapo December 17, 2023 04:36
@Pil0tXia
Copy link
Member Author

@mxsm PTAL~

@pandaapo pandaapo merged commit d66ebe6 into apache:master Dec 17, 2023
@Pil0tXia Pil0tXia deleted the pil0txia_enhance_4665 branch January 4, 2024 05:00
@xwm1992 xwm1992 added this to the 1.11.0 milestone Dec 18, 2024
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.

[Enhancement] Unify 'LiteMessage' to 'EventMeshMessage'

3 participants