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

feat: 新增黑名单用户及VIP用户处理逻辑,同时修改ALLOW_USERS的校验为userid(SenderStaffId),将用户相关校验统一为userid #190

Merged
merged 20 commits into from
Apr 9, 2023

Conversation

ffinly
Copy link
Contributor

@ffinly ffinly commented Apr 9, 2023

在提出此拉取请求时,我确认了以下几点(请复选框):

  • 我已阅读并理解贡献者指南
  • 我已检查没有与此请求重复的拉取请求。
  • 我已经考虑过,并确认这份呈件对其他人很有价值。
  • 我接受此提交可能不会被使用,并根据维护人员的意愿关闭拉取请求。

新增黑名单用户处理逻辑,如用户在黑名单,将禁止在单聊、群聊中使用。同时优化相关答复信息展示。
在配置max_request的前提下,管理员和VIP用户不受对话限制,同时规范用户相关校验,统一为userid,后续新增权限控制也建议采用唯一可确定值

@ffinly ffinly changed the title perf: 新增VIP用户处理逻辑(管理员和VIP用户不受请求次数限制),同时修改ALLOW_USERS的校验为userid(Sender… …StaffId),将用户相关校验统一为userid perf: 新增黑名单用户及VIP用户处理逻辑,同时修改ALLOW_USERS的校验为userid(Sender… …StaffId),将用户相关校验统一为userid Apr 9, 2023
Copy link
Contributor Author

@ffinly ffinly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already Reviewed

@ffinly ffinly changed the title perf: 新增黑名单用户及VIP用户处理逻辑,同时修改ALLOW_USERS的校验为userid(Sender… …StaffId),将用户相关校验统一为userid perf: 新增黑名单用户及VIP用户处理逻辑,同时修改ALLOW_USERS的校验为userid(SenderStaffId),将用户相关校验统一为userid Apr 9, 2023
@ffinly ffinly changed the title perf: 新增黑名单用户及VIP用户处理逻辑,同时修改ALLOW_USERS的校验为userid(SenderStaffId),将用户相关校验统一为userid feat: 新增黑名单用户及VIP用户处理逻辑,同时修改ALLOW_USERS的校验为userid(SenderStaffId),将用户相关校验统一为userid Apr 9, 2023
@eryajf eryajf added the feature label Apr 9, 2023
@eryajf eryajf merged commit 4a29f1e into eryajf:main Apr 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants