From 6c7e49d09357b19058dba6b084e49dbf640b27eb Mon Sep 17 00:00:00 2001
From: sstent <stuart.stent@gmail.com>
Date: Fri, 26 Sep 2025 09:25:07 -0700
Subject: [PATCH] sync - tui loads but no data in views

---
 main.py               | 10 +++++++++-
 tui/views/workouts.py | 25 ++++++-------------------
 2 files changed, 15 insertions(+), 20 deletions(-)

diff --git a/main.py b/main.py
index 1289b37..5ee63f0 100644
--- a/main.py
+++ b/main.py
@@ -14,6 +14,7 @@ from textual.widgets import (
     Header, Footer, Static, Button, DataTable, 
     Placeholder, TabbedContent, TabPane
 )
+from textual import on
 from textual.logging import TextualHandler
 
 from backend.app.config import settings
@@ -165,7 +166,14 @@ class CyclingCoachApp(App):
             for nav_button in self.query("Button"):
                 nav_button.remove_class("-active")
             event.button.add_class("-active")
-    
+
+    @on(TabbedContent.TabActivated)
+    async def on_tab_activated(self, event: TabbedContent.TabActivated) -> None:
+        """Handle tab activation to load data for the active tab."""
+        if event.pane.id == "workouts-tab":
+            workout_view = self.query_one("#workout-view", WorkoutView)
+            workout_view.load_data()
+
     def action_quit(self) -> None:
         """Quit the application."""
         self.exit()
diff --git a/tui/views/workouts.py b/tui/views/workouts.py
index ea8f631..b5d11fb 100644
--- a/tui/views/workouts.py
+++ b/tui/views/workouts.py
@@ -13,7 +13,6 @@ from textual.widgets import (
 from textual.widget import Widget
 from textual.reactive import reactive
 from textual.message import Message
-from textual import on
 from typing import List, Dict, Optional
 
 from backend.app.database import AsyncSessionLocal
@@ -306,23 +305,11 @@ Elevation Gain: {workout.get('elevation_gain_m', 'N/A')} m
         """Load workout data when mounted."""
         self.loading = True
 
-    async def load_workouts_data(self) -> None:
-        """Load workout data and handle the result."""
+    def load_data(self) -> None:
+        """Public method to trigger data loading for the workout view."""
         self.loading = True
         self.refresh()
-        try:
-            workouts, sync_status = await self._load_workouts_data()
-            self.on_workouts_loaded((workouts, sync_status))
-        except Exception as e:
-            self.log(f"Error loading workouts data: {e}", severity="error")
-            self.loading = False
-            self.refresh()
-
-    @on(TabbedContent.TabActivated)
-    async def on_tab_activated(self, event: TabbedContent.TabActivated) -> None:
-        """Handle tab activation to load data for the active tab."""
-        if event.pane.id == "workouts-tab":
-            await self.load_workouts_data()
+        self.run_async(self._load_workouts_data(), self.on_workouts_loaded)
 
     async def _load_workouts_data(self) -> tuple[list, dict]:
         """Load workouts and sync status (async worker)."""
@@ -340,10 +327,10 @@ Elevation Gain: {workout.get('elevation_gain_m', 'N/A')} m
 
     def on_workouts_loaded(self, result: tuple[list, dict]) -> None:
         """Handle loaded workout data."""
-        print("Entering on_workouts_loaded")
+        self.log("Entering on_workouts_loaded")
         try:
             workouts, sync_status = result
-            print(f"on_workouts_loaded received: {len(workouts)} workouts, sync status: {sync_status}")
+            self.log(f"on_workouts_loaded received: {len(workouts)} workouts, sync status: {sync_status}")
             self.workouts = workouts
             self.sync_status = sync_status
             self.loading = False
@@ -351,7 +338,7 @@ Elevation Gain: {workout.get('elevation_gain_m', 'N/A')} m
             self.populate_workouts_table()
             self.update_sync_status()
         except Exception as e:
-            print(f"Error in on_workouts_loaded: {e}")
+            self.log(f"Error in on_workouts_loaded: {e}", severity="error")
             self.loading = False
             self.refresh()