-
Notifications
You must be signed in to change notification settings - Fork 45
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 wx-work bot template #84
Conversation
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.
Please fix your code so that we can merge it.
src/issue-handlers.ts
Outdated
@@ -183,6 +183,11 @@ async function manageIssue ( | |||
'issue card for chatops', | |||
) | |||
|
|||
if (owner === 'juzibot' && repository === 'Juzi-WeChat-Work-Tasks') { | |||
const wxBotUrl = 'https://qyapi.weixin.qq.com/cgi-bin/webhook/send?key=974db6af-6b24-41aa-8da6-5ed634d24fcf' | |||
require(wxBotUrl) |
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.
I believe here should be request
instead of require
.
package.json
Outdated
@@ -28,6 +28,7 @@ | |||
}, | |||
"homepage": "https://github.com/kaiyuanshe/OSSChat#readme", | |||
"dependencies": { | |||
"@types/request": "^2.48.4", |
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.
Please move @types
to dev dependencies.
Thanks!
Terrific! We start supporting WXWorks from today, thanks jiarui! |
I'm submitting a...
Related to #85
Checklist
Description
please describe the changes that you are making
for features, please describe how to use the new feature
WeChat Work provides a room bot, using a webhook, it can push notification to WeChat work room. This commit helps us to push GitHub notifications to WeChat work room.
please include a reference to an existing issue, if applicable
Does this PR introduce a breaking change?