-
Notifications
You must be signed in to change notification settings - Fork 0
Feat/core refinement #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Refactored study session API to support subject-level and lesson-level review. - Updated database schema for elective lessonId and new subjectId in study_sessions. - Enhanced gamification persistence with server-side XP and streak calculations. - Improved kurama-admin lesson detail page with card list, CRUD, and bulk JSON import. - Fixed numerous TypeScript and linting errors across the monorepo. - Updated CLAUDE.md with new project guidelines and workflows.
- Created SM-2 utility for review interval calculations - Integrated SM-2 into study sessions (lesson and quick review) - Added 'Due Today' section to student dashboard - Enhanced parent dashboard with subject performance grid - Implemented academic excellence alerts for parents - Updated database schema with unique constraint and indexes for user progress - Fixed type errors and improved error handling in server functions
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis PR implements core refinements across admin and user applications, introducing admin access enforcement via middleware, comprehensive card management for lesson administration, persistence mechanisms for parent alerts and achievement notifications, subject-based study sessions with SM-2 spaced repetition tracking, and public legal pages with updated navigation. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant API as Study API
participant DB as Database
Client->>API: POST /api/study/session (subjectId)
activate API
API->>DB: Fetch subject
DB-->>API: Subject data
API->>DB: Fetch all lessons for subject
DB-->>API: Lesson IDs
API->>DB: Fetch cards where lessonId IN (lesson set)
DB-->>API: Cards (up to 50)
API->>DB: Create study session record<br/>(userId, subjectId, mode, startedAt)
DB-->>API: Session ID
deactivate API
API-->>Client: { sessionId, cards[] }
Note over Client: User answers cards
Client->>API: POST /api/study/session/complete<br/>(sessionId, cardResults[])
activate API
API->>DB: Bulk upsert userProgress<br/>with SM-2 calculations
DB-->>API: Updated progress records
deactivate API
API-->>Client: { success: true }
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Poem
✨ Finishing touches
📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (37)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary by CodeRabbit
Release Notes
New Features
Improvements
✏️ Tip: You can customize this high-level summary in your review settings.