KimigaiiWuyi / GenshinUID

支持OneBot(QQ)、OneBotV12、QQ频道、微信、KOOK(开黑啦)、Telegram(电报)、FeiShu(飞书)、DoDo、Villa(米游社大别野)、Discord的全功能HoshinoBot/NoneBot2/Koishi/yunzai/ZeroBot原神机器人插件
https://docs.sayu-bot.com
GNU General Public License v3.0
738 stars 102 forks source link

【bug?】强制推送体力提醒会走正常的逻辑 #647

Open mengluo04 opened 1 month ago

mengluo04 commented 1 month ago

不满足推送条件时不会执行,失去了强制的意义

KimigaiiWuyi commented 1 month ago

应该已更正

mengluo04 commented 1 month ago

应该已更正

测试没有,看了代码,get_notice_list,然后调用了all_check->checkcheck判断不满足条件就给all_check返回false了,返回的msg_list是空对象 image

KimigaiiWuyi commented 1 month ago

你说的意思是无论阈值多少,都给全部玩家推送体力状态? 这个功能不是强制推送体力提醒预期干的事情

mengluo04 commented 1 month ago

你说的意思是无论阈值多少,都给全部玩家推送体力状态? 这个功能不是强制推送体力提醒预期干的事情

我理解的强制推送是这个意思,我理解偏差了。 预期的功能是? 现在是满足条件自动推送之后也不会强制推送,那应该什么条件才会推送?满足条件但是没有自动推送的情况?

KimigaiiWuyi commented 1 month ago

你说的意思是无论阈值多少,都给全部玩家推送体力状态? 这个功能不是强制推送体力提醒预期干的事情

我理解的强制推送是这个意思,我理解偏差了。 预期的功能是? 现在是满足条件自动推送之后也不会强制推送,那应该什么条件才会推送?满足条件但是没有自动推送的情况?

意思是不到半小时也进行一次推送, 用于测试. 如果你需要无论阈值多少,都给全部玩家推送体力状态这个功能的话 请阐述一下使用场景, 我可以视情况加入

mengluo04 commented 1 month ago

你说的意思是无论阈值多少,都给全部玩家推送体力状态? 这个功能不是强制推送体力提醒预期干的事情

我理解的强制推送是这个意思,我理解偏差了。 预期的功能是? 现在是满足条件自动推送之后也不会强制推送,那应该什么条件才会推送?满足条件但是没有自动推送的情况?

意思是不到半小时也进行一次推送, 用于测试. 如果你需要无论阈值多少,都给全部玩家推送体力状态这个功能的话 请阐述一下使用场景, 我可以视情况加入

倒也没有啥使用场景,只是看到这个功能按我的理解测试了一下,忽略这个需求吧

mengluo04 commented 1 month ago

你说的意思是无论阈值多少,都给全部玩家推送体力状态? 这个功能不是强制推送体力提醒预期干的事情

我理解的强制推送是这个意思,我理解偏差了。 预期的功能是? 现在是满足条件自动推送之后也不会强制推送,那应该什么条件才会推送?满足条件但是没有自动推送的情况?

意思是不到半小时也进行一次推送, 用于测试. 如果你需要无论阈值多少,都给全部玩家推送体力状态这个功能的话 请阐述一下使用场景, 我可以视情况加入

有没有考虑把定时推送规则做成配置,自己写cron表达式或者用选择的形式来设置

KimigaiiWuyi commented 1 month ago

你说的意思是无论阈值多少,都给全部玩家推送体力状态? 这个功能不是强制推送体力提醒预期干的事情

我理解的强制推送是这个意思,我理解偏差了。 预期的功能是? 现在是满足条件自动推送之后也不会强制推送,那应该什么条件才会推送?满足条件但是没有自动推送的情况?

意思是不到半小时也进行一次推送, 用于测试. 如果你需要无论阈值多少,都给全部玩家推送体力状态这个功能的话 请阐述一下使用场景, 我可以视情况加入

有没有考虑把定时推送规则做成配置,自己写cron表达式或者用选择的形式来设置

这个可以有, 之后加入吧 最近确实有点忙就是了

mengluo04 commented 1 month ago

你说的意思是无论阈值多少,都给全部玩家推送体力状态? 这个功能不是强制推送体力提醒预期干的事情

我理解的强制推送是这个意思,我理解偏差了。 预期的功能是? 现在是满足条件自动推送之后也不会强制推送,那应该什么条件才会推送?满足条件但是没有自动推送的情况?

意思是不到半小时也进行一次推送, 用于测试. 如果你需要无论阈值多少,都给全部玩家推送体力状态这个功能的话 请阐述一下使用场景, 我可以视情况加入

有没有考虑把定时推送规则做成配置,自己写cron表达式或者用选择的形式来设置

这个可以有, 之后加入吧 最近确实有点忙就是了

不着急,大佬你先忙