Skip to content

feat(admin): learning effectiveness KPIs with tester exclusion - #46

Open
prior89 wants to merge 1 commit into
mainfrom
feature/learning-metrics-kpi
Open

prior89 wants to merge 1 commit into
mainfrom
feature/learning-metrics-kpi

Conversation

@prior89

@prior89 prior89 commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Summary

Admin learning effectiveness KPI dashboard with tester account exclusion.

Changes

  • learning-metrics.service.ts — D1/D7 retention, mastery velocity, review completion, solver→practice funnel
  • metrics-test-user-filter.ts — excludes @test.mathema, +test emails, TESTER role
  • AdminLearningMetricsScreen.tsx — Korean UI with tester exclusion notice
  • Client learning event instrumentation (Study/Review/Notes/Upload)
  • verify-learning-kpi.ts script

Test plan

  • npm run typecheck
  • npx jest learning-metrics.service.spec.ts metrics-test-user-filter.spec.ts
  • Admin KPI screen loads for ADMIN role
  • Tester accounts excluded from aggregates

North Star, retention, mastery, Error DNA aggregates; admin Korean UI; learning events instrumentation; metrics-test-user-filter; verify-learning-kpi script.
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.

1 participant