Reactor migration from XAML/MVVM#2
Merged
davidortinau merged 34 commits intomainfrom Mar 29, 2025
Merged
Conversation
…agement in DashboardPage
…ate management; update vocabulary and skill profile selection handling.
…ipboard functionality; introduce MessageTemplateSelector for message display logic.
…e management for improved user interaction
… OpenAI package and enhance conversation handling
…mplement mode selector component for user interaction
…ns; improve user interaction and state management
…hance auto-transition functionality and error handling for loading more sentences
…Shell and MauiProgram
…gram and AppShell
…or; update EditVocabularyPage to set ListId from fetched vocabulary list
…to utilize it; localize OK string
…pdate language selection handling
…ent; implement IAppState for user profile management
…lection; update DashboardPage to include DescribeAScene route
…ndling and popup management
…g; add resource styles for consistency
…r image sources; update image aspect settings for better display
…nused properties and comments
…cationTheme for consistency; update input handling in ModeSelector
… remove unused properties for cleaner code
…ndling; update GradeResponse model with descriptions and clean up unused code
…roject file comments
…cribeAScenePage and WritingPage
…unctionality and dependency management
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
1 participant
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.