Skip to content

Refocus trainer dashboard on actionable stats - #64

Merged
OwenTanzer merged 2 commits into
mainfrom
claude/next-stage-project-status-m163fn
Jul 22, 2026
Merged

Refocus trainer dashboard on actionable stats#64
OwenTanzer merged 2 commits into
mainfrom
claude/next-stage-project-status-m163fn

Conversation

@OwenTanzer

Copy link
Copy Markdown
Owner

Summary

  • Move Needs Attention and Recently Worked to the top of the trainer dashboard, right after the header — these are the stats a trainer actually uses to decide what to do next.
  • Remove the Career Highlights section entirely (training logs, logs this week/month, milestones completed, skills worked on total).
  • Remove the Skills You've Worked Most section entirely.
  • Replace the retrospective "A look back at your training career..." copy with a direct, operational description of the page.
  • Remove the now-dead computations backing the removed sections (totalLogs, logsThisWeek, logsThisMonth, milestonesCompleted, skillsWorkedOnTotal, mostWorkedSkills, the SkillWorkedCount type, and the now-unused daysAgoLocalDate helper) from useTrainerHistoryStats.

Your Dogs, Success Rate, and Final Evaluation Outcomes are unchanged in content, just reordered below the actionable sections.

Why

Issue #27's follow-up scope asked for a prioritization/subtraction pass on the existing stats page — put actionable statistics first, drop the decorative career-retrospective sections, and don't replace them with equally decorative summaries. This PR does exactly that without adding any new analytics.

Validation

  • npm run build — production TypeScript and Vite build passed
  • npm test — 28 tests passed
  • npm run lint — passed with the same 3 pre-existing warnings as main (unrelated to this change)

Closes #27


Generated by Claude Code

Move Needs Attention and Recently Worked to the top of the page since
those are what a trainer uses to decide what to do next. Remove the
decorative Career Highlights and Skills You've Worked Most sections
and their backing computations, and drop the retrospective "look back
at your training career" framing for a direct description.

Closes #27

Copy link
Copy Markdown
Owner Author

One small behavioral mismatch remains before merge: the follow-up scope asks for refined/actionable statistics to appear first, but TrainerHistory still initializes refinedRate to false, so Overall is the default success-rate view. Please change the initial state to true (and preferably place the Refined button before Overall so the visual order matches the default). After that, this looks ready to merge.

Address review feedback on PR #64: the Refined view (which omits
dogs marked Excluded from Stats) is the more actionable of the two
success-rate views, so it should be the default and appear first
among the toggle buttons, matching the dashboard's actionable-first
ordering.
@OwenTanzer
OwenTanzer merged commit 3722858 into main Jul 22, 2026
1 check passed
@OwenTanzer
OwenTanzer deleted the claude/next-stage-project-status-m163fn branch July 22, 2026 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add an instructor stats page/dashboard

2 participants