Fix build for Windows#1
Closed
mattleibow wants to merge 8 commits intodavidortinau:mainfrom
Closed
Conversation
davidortinau
added a commit
that referenced
this pull request
Nov 20, 2025
**Critical Fix for Both Issues:** Issue #1 - "No cached plan found" preventing progress save: - UpdatePlanItemProgressAsync now updates database FIRST - No longer requires cache to exist - Progress saves even if cache is lost Issue #2 - New plan generated on Dashboard return: - InitializePlanCompletionRecordsAsync pre-creates all DailyPlanCompletion records - Called immediately after plan generation - Ensures database has complete plan structure - EnrichPlanWithCompletionDataAsync can now restore progress **How it works:** 1. Plan generated → DailyPlanCompletion records created with 0 minutes 2. User practices → UpdatePlanItemProgressAsync updates DB (cache optional) 3. Return to Dashboard → GetCachedPlanAsync enriches from DB 4. Same plan IDs preserved → progress retained The cache is now just a performance optimization, not required for correctness.
davidortinau
added a commit
that referenced
this pull request
Nov 20, 2025
**Issue #1: Wrong resource being used** - Plan specified 'General Vocabulary' but activity loaded 'A day in Bali' - Problem: Resource not in selected resources list, fell back to default - Solution: Load resource from database if not in selected list **Issue #2: Activity stalling after answer** - User enters answer, nothing happens, just keeps rendering - Added comprehensive logging to CheckAnswer() to diagnose - Will show exactly where it's failing or hanging Next step: Run the app and review logs to see where CheckAnswer() is stalling.
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
2 participants
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.
No description provided.