-
Notifications
You must be signed in to change notification settings - Fork 308
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 workwechat msgtype #1369
add workwechat msgtype #1369
Conversation
Please add this line to the New features section at the top of
|
already done |
I apologize for the lack of explanation. Please add it below what is currently listed. current
fix
|
Has test code been added when work_wechat_msgtype is set to markdown? |
@@ -795,6 +795,7 @@ properties: | |||
|
|||
### WorkWechat | |||
work_wechat_bot_id: { type: string } | |||
work_wechat_msgtype: { type: string } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If a value other than text or markdown is specified for work_wechat_msgtype in the pyyaml validation check, an exception will occur when the rule is executed.
work_wechat_msgtype: { type: string }
to
work_wechat_msgtype: {type: string, enum: ['text', 'markdown']}
This PR is stale because it has been open for 30 days with no activity. The longer a PR remains stale the more out of date with the main branch it becomes. |
@jertel |
Description
Checklist
make test-docker
with my changes.Questions or Comments