Show recorded scores in past attempt rows

未关闭 适合新手
#604 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
78/100
Issue 类型
功能
描述清晰度
描述清楚
活跃度
活跃
技术栈
javascript, rust
领域
frontend

调研方向

阅读 web/app.js 中的 renderAttemptHistory(),以及 web/progress.js 中由 normalizeProgressEntry() 暴露的已记录字段。添加一个使用合成已保存尝试的浏览器测试,并在编辑器、白板、不完整和旧版条目的报告处于折叠状态时检查行中可见的文本。完成标准是:行显示有效的已记录分数,以及不完整或未评分报告的指定标签。

由索引模型根据 Issue 内容生成。

描述

Motivation

Candidates comparing their results in Interview progress → Past attempts currently have to open each report to see its scores. The row already shows the date, problem, difficulty, language, and verdict. Showing the saved scores alongside the verdict would let candidates compare attempts while their reports are collapsed.

Expected behavior

  • Show the recorded work and communication scores in each attempt row. An editor attempt could read HIRE · Coding 82/100 · Communication 74/100.
  • Label the work score Board work for whiteboard attempts, consistent with the full report.
  • Omit missing scores and display an explicitly recorded zero as 0/100.
  • Show INCOMPLETE without scores for incomplete reports. Show UNSCORED when a saved report has no valid recorded verdict.

Proposed approach

At af66541, renderAttemptHistory() in web/app.js builds the row label, and normalizeProgressEntry() in web/progress.js already exposes the recorded verdict and scores. Using those fields lets the row distinguish missing values from recorded zeroes. A separate comparison panel would require another interaction, so I propose extending the existing row.

The change is limited to the Past attempts list. Add a browser test with synthetic saved attempts that checks the visible row text while the reports are collapsed, including editor, whiteboard, incomplete, and legacy entries.

Related work

#276 presents phase scores in charts, and #466 compares attempts inside an opened report. The CodeTrial contribution guide lists attempt history with scores in the HackerRank Mock Interviews comparison.

主要语言
Rust
星标
223
派生
148
平均合并
1 天 10 小时
30 天内合并 PR
227

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

sysprog21/codetrial 的其他 Issue

查看 sysprog21/codetrial 的全部 Issue

相似的 Issue

更多 Rust Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。