You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@answer.apache.org by "lhui (via GitHub)" <gi...@apache.org> on 2025/01/03 04:15:48 UTC
[PR] create notification wecom plugins [incubator-answer-plugins]
lhui opened a new pull request, #270:
URL: https://github.com/apache/incubator-answer-plugins/pull/270
Hi answer plugins team,
I have created a WeCom notification plugin for the Answer application based on the notification-dingtalk plugin. Could you please help me merge it?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@answer.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] create notification wecom plugins [incubator-answer-plugins]
Posted by "LinkinStars (via GitHub)" <gi...@apache.org>.
LinkinStars commented on PR #270:
URL: https://github.com/apache/incubator-answer-plugins/pull/270#issuecomment-2568680877
@lhui Great. Thank you for your contribution, there are a few small issues need to be fixed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@answer.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] create notification wecom plugins [incubator-answer-plugins]
Posted by "LinkinStars (via GitHub)" <gi...@apache.org>.
LinkinStars commented on code in PR #270:
URL: https://github.com/apache/incubator-answer-plugins/pull/270#discussion_r1901457252
##########
notification-wecom/notification_wecom.go:
##########
@@ -0,0 +1,145 @@
+package wecom
Review Comment:
You need to add the ASF(Apache Software Foundation) header.
##########
notification-wecom/README_CN.md:
##########
@@ -0,0 +1,33 @@
+# 企业微信通知插件
+
+## 使用指南
+
+要使用通知企业微信通知插件，请使用以下命令进行安装：
+
+```bash
+./answer build --with github.com/apache/incubator-answer-plugins/notification-wecom
+```
+
+
+## 功能
+
+- 通过企业微信群机器人发送信息到企业微信
+
+## 配置
+
+> 配置Webhook URL并打开通知
+
+- Webhook 地址: 类似于 `https://qyapi.weixin.qq.com/cgi-bin/webhook/send?key=xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx`
+
+## 预览
+
+
+
+## 开发文档
+
+https://open.work.weixin.qq.com/help2/pc/14931
+https://developer.work.weixin.qq.com/document/path/91770
+
+## 致谢
+
+感谢钉钉通知插件的开发，本插件很多代码参考壶或者直接沿用了钉钉通知插件的代码。
Review Comment:
remove `壶`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@answer.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] create notification wecom plugins [incubator-answer-plugins]
Posted by "LinkinStars (via GitHub)" <gi...@apache.org>.
LinkinStars merged PR #270:
URL: https://github.com/apache/incubator-answer-plugins/pull/270
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@answer.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] create notification wecom plugins [incubator-answer-plugins]
Posted by "LinkinStars (via GitHub)" <gi...@apache.org>.
LinkinStars commented on code in PR #270:
URL: https://github.com/apache/incubator-answer-plugins/pull/270#discussion_r1901457761
##########
notification-wecom/go.mod:
##########
@@ -0,0 +1,47 @@
+module github.com/apache/incubator-answer-plugins/notification-wecom
+
+go 1.23.4
Review Comment:
Use the `go 1.22.0` version that is the same as [other plugins](https://github.com/apache/incubator-answer-plugins/blob/d99270e8823a7a18206b00da40d6035fa85ca082/captcha-google-v2/go.mod#L3C1-L3C10).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@answer.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] create notification wecom plugins [incubator-answer-plugins]
Posted by "lhui (via GitHub)" <gi...@apache.org>.
lhui commented on PR #270:
URL: https://github.com/apache/incubator-answer-plugins/pull/270#issuecomment-2568720200
Hi, I have fixed above and recheck code, please help me merge it. Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@answer.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] create notification wecom plugins [incubator-answer-plugins]
Posted by "lhui (via GitHub)" <gi...@apache.org>.
lhui commented on PR #270:
URL: https://github.com/apache/incubator-answer-plugins/pull/270#issuecomment-2568685509
Thanks, I will fix above and push it later.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@answer.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org