search

lddsb / drone-dingtalk-message

A plugins for drone ci to message a group by robot
MIT License
63 stars 31 forks source link

atMobile没有实现传入吗 #26

Closed LGinC closed 3 years ago

LGinC commented 4 years ago

image

lddsb commented 3 years ago

这个暂时没有实现，需要at某个人的时候感觉操作比较繁琐，如果有什么好的想法欢迎pr

LGinC commented 3 years ago

奇怪，你这里明明都在content里追加了@手机号了，实际效果确是没有@到 https://github.com/lddsb/dingtalk-webhook/blob/c98992940848434092a7f896147753aafdbe5785/webhook.go#L189

按理说只要在.drone.yml里配置msg_at_mobiles 应该就可以才对呀

lddsb commented 3 years ago

奇怪，你这里明明都在content里追加了@手机号了，实际效果确是没有@到 https://github.com/lddsb/dingtalk-webhook/blob/c98992940848434092a7f896147753aafdbe5785/webhook.go#L189

按理说只要在.drone.yml里配置msg_at_mobiles 应该就可以才对呀

我今天抽时间改一下吧，这个功能是刚需吗？

LGinC commented 3 years ago

奇怪，你这里明明都在content里追加了@手机号了，实际效果确是没有@到 https://github.com/lddsb/dingtalk-webhook/blob/c98992940848434092a7f896147753aafdbe5785/webhook.go#L189 按理说只要在.drone.yml里配置msg_at_mobiles 应该就可以才对呀

我今天抽时间改一下吧，这个功能是刚需吗？

是的, 不同项目需要通知不同的人

好像是手机号正则的问题,对新号段的匹配不到 (13\d|14[579]|15[^4\D]|17[^49\D]|18\d)\d{8} 可以试试这个,这个范围更广一点

LGinC commented 3 years ago

提交PR了

lddsb commented 3 years ago

34 已修复艾特不到人的BUG

请更新到 1.2.7

可以查看一下README参考用法