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

[ISSUE #4665] Optimize and sync recent new connectors' docs #4668

Merged
merged 12 commits into from
Dec 17, 2023

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
```yaml
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
11 checks passed
@Pil0tXia Pil0tXia deleted the pil0txia_enhance_4665 branch January 4, 2024 05:00
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