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.

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

禁言功能存在 bug,使用禁 @somebody时无法工作 #15

Closed
tom-snow opened this issue May 29, 2022 · 8 comments
Closed

禁言功能存在 bug,使用禁 @somebody时无法工作 #15

tom-snow opened this issue May 29, 2022 · 8 comments
Labels
bug 一起抓虫子 产品经理 人人都是产品经理

Comments

@tom-snow
Copy link
Collaborator

问题代码:

if len(msg.split()) > len(sb):

应改为:

if len(msg.split()) > len(sb)+1:

另外建议词云功能也像违禁词一样使用资源配置文件,并且考虑再创建一个 "custom" 文件来存储用户自定义的违禁词/词云忽略词,而且可以加个指令清除当前群的词云存储文件

@yzyyz1387
Copy link
Owner

问题一,代码没问题
问题二,日后考虑

@yzyyz1387
Copy link
Owner

问题一示例
image

@yzyyz1387
Copy link
Owner

+1是a16的写法
image

@yzyyz1387
Copy link
Owner

你加空格了是吧,禁打完直接@不是方便点吗,后期再看看,把消息的空格处理了啥的

@tom-snow
Copy link
Collaborator Author

另外

await ban.finish("权限不足")

这里的 ban 是否应该对应改成 unban ?后面还有很多类似的

@yzyyz1387
Copy link
Owner

确实,要不你pr一下,最近很忙

@yzyyz1387
Copy link
Owner

其实也不影响使用,所以没发现😂

@yzyyz1387 yzyyz1387 added the bug 一起抓虫子 label May 30, 2022
@yzyyz1387
Copy link
Owner

问题代码:

if len(msg.split()) > len(sb):

应改为:

if len(msg.split()) > len(sb)+1:

另外建议词云功能也像违禁词一样使用资源配置文件,并且考虑再创建一个 "custom" 文件来存储用户自定义的违禁词/词云忽略词,而且可以加个指令清除当前群的词云存储文件

TODO

@yzyyz1387 yzyyz1387 added the 产品经理 人人都是产品经理 label May 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 一起抓虫子 产品经理 人人都是产品经理
Projects
None yet
Development

No branches or pull requests

2 participants