Skip to content

chore(release): 发布已解散社团身份修复 - #241

Merged
Palind-Rome merged 3 commits into
mainfrom
dev
Sep 11, 2026
Merged

Palind-Rome merged 3 commits into
mainfrom
dev

Conversation

@Palind-Rome

Copy link
Copy Markdown
Owner

改动内容

  • 将已解散社团的社团级角色从认证会话和权限角色汇总中排除。
  • 权限快照命中缓存后按当前社团状态复核,避免解散后继续使用过期的社团权限。
  • 保留历史成员、任期和 USER_ROLES 数据,未增加数据库迁移。

改动原因

发布 PR #240 在 dev 已完成验证,修复运营工作台仍显示“测试社团成员/负责人”的问题,并保证解散社团的历史记录可追溯。


关联 Issue

Part of #237

审查关注点

  • dev 中的认证角色过滤和权限缓存复核是否完整进入阶段性稳定版本。
  • 不删除历史成员或角色记录,不改变数据库结构。

UI 改动

普通用户重新加载或刷新会话后,工作台身份卡片和顶部角色提示不再显示已解散社团身份。

测试说明

  • dev PR fix(auth): 隐藏已解散社团的当前身份 #240 的 CI、代码质量门禁全部通过。
  • 后端本地测试:311 个通过。
  • 回归测试验证运营社团角色保留、已解散社团角色排除、历史 UserRole 记录保留和缓存角色过滤。

检查清单

代码质量

  • 后端代码已构建通过
  • 前端代码已构建通过(本发布仅含后端改动)
  • 已完成本地回归测试

数据库(仅涉及时勾选)

  • 无表结构变化
  • 表结构变更已写入 database/
  • 种子数据、视图、索引、触发器、存储过程已同步

文档与部署(仅涉及时勾选)

  • 课程文档已同步更新
  • 需要新增或修改 GitHub Secrets

* 做了什么:认证角色汇总仅保留运营中且审核通过社团的社团级角色。\n* 为什么:解散社团的历史成员和角色需要保留,但不应继续显示为当前身份或参与权限判断。\n* 影响范围:登录与会话角色、权限角色以及社团可见性回归测试。\n* 注意事项:不修改数据库历史记录,也不需要迁移。
* 做了什么:权限快照读取后再次按当前社团状态过滤,并补充缓存角色回归测试。\n* 为什么:社团解散后已有权限快照不能在缓存 TTL 内继续授予社团范围权限。\n* 影响范围:权限检查和社团可见性测试。\n* 注意事项:保留历史 UserRole 记录,不需要数据库迁移。
合并已验证:CI、代码质量门禁和 311 项后端测试均通过。
@Palind-Rome Palind-Rome added bug Something isn't working 优先级:P1 核心功能任务 area:auth 用户、登录、注册、角色权限 area:club 社团组织、成员、干部换届 labels Sep 11, 2026
@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 40 minutes.

Check out review usage here.

View limit details

Limit 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.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: d70942ad-6565-4246-af98-a44e4dc2867c

📥 Commits

Reviewing files that changed from the base of the PR and between dfdbcd2 and 55503a3.

📒 Files selected for processing (2)
  • backend.Tests/ClubVisibilityTests.cs
  • backend/Services/AuthService.cs

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Palind-Rome
Palind-Rome merged commit 2b2b507 into main Sep 11, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:auth 用户、登录、注册、角色权限 area:club 社团组织、成员、干部换届 bug Something isn't working 优先级:P1 核心功能任务

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant