reviewGrades and manageSubmissions rosters don't filter by role
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 68/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
- 技术栈
- javascript, python
- 领域
- frontend
调研方向
从 frontend/src/pages/reviewGrades/index.js 和 frontend/src/pages/manageSubmissions/index.js 入手，然后追踪 get_course_enrollment 和课程角色是如何提供给这些页面的。让两个名单都排除其他 TA 和 co-instructor，同时保留学生；当工作人员不再出现在任一名单中时，即为完成。
由索引模型根据 Issue 内容生成。
描述
get_course_enrollment returns all enrolled users regardless of role. Neither frontend/src/pages/reviewGrades/index.js (which only excludes the current viewer: allRes.filter(s => s.id !== userInfo.id)) nor frontend/src/pages/manageSubmissions/index.js (no filtering at all) excludes other TAs or co-instructors from the roster they display. If a TA or a second instructor is enrolled in a course and has ever had a Submission row (e.g. via future staff-testing, see #363), they'd currently show up alongside real students in grade review and submission management.
This is independent of whether staff-testing (#363) ever ships — it's a pre-existing gap in role filtering on these two pages today.
Found while investigating a review comment from Cecilia (yczhangcc) on PR #355.
- 主要语言
- Python
- 星标
- 13
- 派生
- 0
- 平均合并
- 9 天 7 小时
- 30 天内合并 PR
- 5
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
kiat/codeAssist 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 68/100
kiat/codeAssist#391 ·
-
Grading follow-ups from PR #394可能已有人在做 @DarioDeSilva 于 10 天前认领。 未关闭
kiat/codeAssist#397 · 已指派 1 人 ·
-
难度 3/5 1-2 天 新手友好度 42/100
kiat/codeAssist#393 ·
-
难度 3/5 1-2 天 新手友好度 63/100
kiat/codeAssist#392 ·
-
难度 4/5 3-5 天 新手友好度 52/100
kiat/codeAssist#390 ·
相似的 Issue
-
[BUG]: Compliance rules never see Unity Catalog views as views可能已有人在做 @wagnercsantos 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 78/100
databrickslabs/ontos#1348 ·
维护者通常 3 天内回复
-
`forum_pr_poster` action will always return `none` when checking for pre-existing feedback threads, even if a feedback thread already exists.可能已有人在做 @pgmzeta 今天认领。 未关闭E-Verified P-Major
难度 2/5 1-3 小时 新手友好度 79/100
goonstation/goonstation#28259 ·
维护者通常 2 天内回复
-
enhancement
难度 2/5 1-3 小时 新手友好度 65/100
woonstadrotterdam/monumenten#70 ·
维护者通常 15 天内回复
-
binding:python bug
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
The Browser tab fails with a Wayland error when Open Terminal runs from a Wayland desktop session未关闭
难度 2/5 1-3 小时 新手友好度 78/100
open-webui/open-terminal#210 ·