Repository navigation
chore(release): 发布已解散社团身份修复 - #241
Merged
Merged
Conversation
* 做了什么:认证角色汇总仅保留运营中且审核通过社团的社团级角色。\n* 为什么:解散社团的历史成员和角色需要保留,但不应继续显示为当前身份或参与权限判断。\n* 影响范围:登录与会话角色、权限角色以及社团可见性回归测试。\n* 注意事项:不修改数据库历史记录,也不需要迁移。
* 做了什么:权限快照读取后再次按当前社团状态过滤,并补充缓存角色回归测试。\n* 为什么:社团解散后已有权限快照不能在缓存 TTL 内继续授予社团范围权限。\n* 影响范围:权限检查和社团可见性测试。\n* 注意事项:保留历史 UserRole 记录,不需要数据库迁移。
合并已验证:CI、代码质量门禁和 311 项后端测试均通过。
Palind-Rome
requested review from
Aphrody-Dy,
Chaotze,
FEnjoyit,
SE-MAX-F,
YoumiaHina,
chiyuuu0123,
jyybacpg,
onnisama and
wendaining
as code owners
September 11, 2026 07:52
Contributor
|
Warning Review limit reachedNext included review available in 40 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
改动内容
USER_ROLES数据,未增加数据库迁移。改动原因
发布 PR #240 在
dev已完成验证,修复运营工作台仍显示“测试社团成员/负责人”的问题,并保证解散社团的历史记录可追溯。关联 Issue
Part of #237
审查关注点
dev中的认证角色过滤和权限缓存复核是否完整进入阶段性稳定版本。UI 改动
普通用户重新加载或刷新会话后,工作台身份卡片和顶部角色提示不再显示已解散社团身份。
测试说明
devPR fix(auth): 隐藏已解散社团的当前身份 #240 的 CI、代码质量门禁全部通过。UserRole记录保留和缓存角色过滤。检查清单
代码质量
数据库(仅涉及时勾选)
database/文档与部署(仅涉及时勾选)