Repository navigation
Practice Mode and Training - #485
Merged
Mkalbani merged 1 commit intoOct 20, 2026
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
@Nacho1499 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
10 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #448
What does this PR do?
Adds a separate practice mode with unlimited attempts, guidance, instant feedback, replay, difficulty adjustment, and non-ranked progress tracking.
Type of change
How was it tested?
npm test -- --runInBand src/game-engine/tests/practice-mode.service.spec.ts— passed (10 tests).npx eslinton the changed files — passed.npx prettier --checkon the changed files — passed.npm run typecheck -- --incremental false --pretty false— failed due to existing type errors in unrelated files; no practice-mode errors were reported.No manual curl verification was performed.
Checklist
[x] I added/updated tests for the change
CONTRIBUTING.mdScreenshots / Logs
Focused unit test result: 1 suite passed, 10 tests passed.
Risk & rollout
Adds practice-session and practice-attempt tables. Apply the included database migration before enabling the practice endpoints. Existing ranked puzzle creation remains unchanged; practice attempts earn no rewards and do not affect ranked progress.