Skip to content
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.

Sign up for GitHub

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

Jump to bottom

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

Open
mengluo04 opened this issue Aug 19, 2024 · 9 comments
Open

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

mengluo04 opened this issue Aug 19, 2024 · 9 comments
Labels
bug?🐛 Something isn't working enhancement✨ New feature or request

Comments

@mengluo04
Copy link
Contributor

不满足推送条件时不会执行，失去了强制的意义
@KimigaiiWuyi KimigaiiWuyi added the bug?🐛 Something isn't working label Aug 20, 2024
KimigaiiWuyi added a commit that referenced this issue Aug 20, 2024
@KimigaiiWuyi
🐛 强制推送体力提醒强制生效 (#647)
c218428
@KimigaiiWuyi
Copy link
Owner

应该已更正

@mengluo04
Copy link
Contributor Author

应该已更正

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

@KimigaiiWuyi
Copy link
Owner

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

@mengluo04
Copy link
Contributor Author

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

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

@KimigaiiWuyi
Copy link
Owner

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

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

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

@mengluo04
Copy link
Contributor Author

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

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

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

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

@mengluo04
Copy link
Contributor Author

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

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

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

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

@KimigaiiWuyi
Copy link
Owner

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

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

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

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

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

@KimigaiiWuyi KimigaiiWuyi added the enhancement✨ New feature or request label Aug 20, 2024
@mengluo04
Copy link
Contributor Author

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

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

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

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

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

不着急，大佬你先忙

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug?🐛 Something isn't working enhancement✨ New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mengluo04 @KimigaiiWuyi