修复确认导入的校验提示及身份重算 #2

2026-10-14T15:23:02+08:00

aerospaceboy commented

2026-10-14 15:23:02 +08:00

目录中混入其他学生材料或身份字段不合法时，确认导入按钮此前只显示为禁用，用户无法判断哪份材料造成冲突；前后端的选中材料校验也不完全一致。本 PR 为每份材料显示封面身份，按未忽略材料重新计算身份，并把所有阻塞原因直接展示在导入页；后端同步拒绝空选择、非法学号、未知类型和重复类型。

例如目录中 4 份报告属于李傲琦、1 份 PPT 属于测试样例时，页面会标出 PPT 的不同身份。将 PPT 设为“忽略”并点击“使用所选资料的封面信息”后，身份会恢复为李傲琦，确认导入按钮可用。

验证：

后端相关测试 74 项通过
前端 Playwright 17 项通过
TypeScript/Vite 构建通过
Docker 真实页面验证通过，未提交实际导入、未触发模型调用

提交按关注点拆分为 4 个小提交，每个提交修改 1–3 个文件。

目录中混入其他学生材料或身份字段不合法时，确认导入按钮此前只显示为禁用，用户无法判断哪份材料造成冲突；前后端的选中材料校验也不完全一致。本 PR 为每份材料显示封面身份，按未忽略材料重新计算身份，并把所有阻塞原因直接展示在导入页；后端同步拒绝空选择、非法学号、未知类型和重复类型。例如目录中 4 份报告属于李傲琦、1 份 PPT 属于测试样例时，页面会标出 PPT 的不同身份。将 PPT 设为“忽略”并点击“使用所选资料的封面信息”后，身份会恢复为李傲琦，确认导入按钮可用。验证： - 后端相关测试 74 项通过 - 前端 Playwright 17 项通过 - TypeScript/Vite 构建通过 - Docker 真实页面验证通过，未提交实际导入、未触发模型调用提交按关注点拆分为 4 个小提交，每个提交修改 1–3 个文件。

aerospaceboy added 4 commits 2026-10-14 15:23:02 +08:00

fix: expose per-file cover identities in import scans 0d6dc4b342

fix: explain import blockers and recompute selected identities 1776b1b50c

fix: validate import identities and selected materials consistently 46c45baaa2

docs: record import confirmation fixes and validation eb8c545090

SkyLiCoder requested changes 2026-10-15 13:59:59 +08:00

SkyLiCoder left a comment

选中不同部分的同学分别创建两个独立评分批次，却不能合并为完整的评分批次，不符合项目的最终目标。建议修改为支持在同一评分批次中单独评分某一部分同学，或单独重新评分某一部分同学，或单独评分某一部分同学某一个阶段（或某些阶段）的报告

This pull request doesn't have enough required approvals yet. 0 of 1 official approvals granted.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u https://git.atgroup.topaerospaceboy/WiseGrade fix/import-confirmation:aerospaceboy-fix/import-confirmation

git checkout aerospaceboy-fix/import-confirmation

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: SkyLiCoder/WiseGrade#2

Allow edits from maintainers