通知功能 - Githubissues

babalae / better-genshin-impact

https://bgi.huiyadan.com/

GNU General Public License v3.0

3.71k stars 246 forks source link

Closed Mr-Quin closed 3 months ago

Mr-Quin commented 4 months ago

draft防merge 试着做了以下我在 #251 提到的通知功能。主要是一些任务耗时较长，通知功能可以优化远程/后台使用的体验，不用隔一段时间手动检查了。架构上应该可以支持添加更多的通知类型目前只能算是个概念，还要对接各种任务模块接触c#以来做的最复杂的东西了，诸位大哥可以帮忙瞅瞅

huiyadanli commented 4 months ago

先切换成review状态，不合并

简单看了下（~~晚上再细看~~

INotifier 希望暴露出原始的通知方法，并和 BetterGI 的任务通知解耦（可以再添加一个层级？）。也许这个包以后可以作为独立的包使用（上传nuget，C# 在这些通知方面好像没看到有通用的类库？
希望通知模块支持图片的入参，可以暂时不实现

Mr-Quin commented 4 months ago

谢谢大哥的建议，refactor了一下 INotificationData和通知模板这块还得琢磨琢磨,先添加了Image property

Lightczx commented 4 months ago

INotificationData和通知模板这块还得琢磨琢磨,先添加了Image property

huiyadanli commented 3 months ago

简单测试了下没有问题，是否可以合并？

huiyadanli commented 3 months ago

还有个问题，默认的Webhook实现是通知到哪的 :joy:

Mr-Quin commented 3 months ago

还有个问题，默认的Webhook实现是通知到哪的 😂

并没有特别的地方，我个人的使用场景是discord和homeassistant，写了个脚本转发