Remove bad ASA external links; fix conditions not loading on startup

- Strip all three unverified external link cards (ASA courses, ASA online,
  Quizlet flashcards) from the Learn tab — asa.com is a Brazilian financial
  site, not the American Sailing Association
- Remove dead openUrl() click handlers and unused Intent/Uri imports from LearnFragment
- Parallelize the two sequential API calls in fetchCurrentConditions so the HUD
  populates in ~half the time (weather + marine fetched concurrently via async/await)
- Add onFailure logging in loadConditions so errors are visible in logcat instead
  of silently swallowed
- Eagerly call loadConditions in the setStyle callback as a startup backstop: the
  camera-idle listener fires before GPS fixes and the initial position may be (0,0),
  so this guarantees a real fetch once the style is ready with a valid camera position

https://claude.ai/code/session_01HXPjBsogsJVRwCiekDGkJX

4 files changed, 16 insertions, 18 deletions

diff --git a/android-app/app/src/main/kotlin/org/terst/nav/MainActivity.kt b/android-app/app/src/main/kotlin/org/terst/nav/MainActivity.kt
index e6355bd..01971a6 100644
--- a/android-app/app/src/main/kotlin/org/terst/nav/MainActivity.kt
+++ b/android-app/app/src/main/kotlin/org/terst/nav/MainActivity.kt
@@ -429,6 +429,13 @@ class MainActivity : AppCompatActivity(), SafetyFragment.SafetyListener {
                 val windArrowBitmap = rasterizeDrawable(R.drawable.ic_wind_arrow)
                 mapHandler?.setupLayers(style, anchorBitmap, arrowBitmap, userBitmap)
                 mapHandler?.setupWindLayer(style, windArrowBitmap)
+                // Eagerly load conditions at style-ready time so the HUD is never blank
+                // on startup — the camera-idle may fire before GPS fixes, this is the backstop.
+                maplibreMap.cameraPosition.target?.let { center ->
+                    if (center.latitude != 0.0 || center.longitude != 0.0) {
+                        viewModel.loadConditions(center.latitude, center.longitude)
+                    }
+                }
             }
 
             maplibreMap.addOnCameraIdleListener {
diff --git a/android-app/app/src/main/kotlin/org/terst/nav/data/repository/WeatherRepository.kt b/android-app/app/src/main/kotlin/org/terst/nav/data/repository/WeatherRepository.kt
index dc47a20..26c5da7 100644
--- a/android-app/app/src/main/kotlin/org/terst/nav/data/repository/WeatherRepository.kt
+++ b/android-app/app/src/main/kotlin/org/terst/nav/data/repository/WeatherRepository.kt
@@ -1,5 +1,7 @@
 package org.terst.nav.data.repository
 
+import kotlinx.coroutines.async
+import kotlinx.coroutines.coroutineScope
 import org.terst.nav.data.api.MarineApiService
 import org.terst.nav.data.api.WeatherApiService
 import org.terst.nav.data.model.ForecastItem
@@ -52,8 +54,11 @@ class WeatherRepository(
      */
     suspend fun fetchCurrentConditions(lat: Double, lon: Double): Result<MarineConditions> =
         runCatching {
-            val weather = weatherApi.getWeatherForecast(lat, lon, forecastDays = 1)
-            val marine = marineApi.getMarineForecast(lat, lon)
+            coroutineScope {
+            val weatherDeferred = async { weatherApi.getWeatherForecast(lat, lon, forecastDays = 1) }
+            val marineDeferred  = async { marineApi.getMarineForecast(lat, lon) }
+            val weather = weatherDeferred.await()
+            val marine  = marineDeferred.await()
             val w = weather.hourly
             val m = marine.hourly
             MarineConditions(
@@ -68,6 +73,7 @@ class WeatherRepository(
                 currentSpeedKt = m.oceanCurrentVelocity.firstOrNull()?.let { it * 1.94384 },
                 currentDirDeg  = m.oceanCurrentDirection.firstOrNull()
             )
+            }
         }
 
     /**
diff --git a/android-app/app/src/main/kotlin/org/terst/nav/ui/MainViewModel.kt b/android-app/app/src/main/kotlin/org/terst/nav/ui/MainViewModel.kt
index 9bd660b..df31949 100644
--- a/android-app/app/src/main/kotlin/org/terst/nav/ui/MainViewModel.kt
+++ b/android-app/app/src/main/kotlin/org/terst/nav/ui/MainViewModel.kt
@@ -127,6 +127,7 @@ class MainViewModel(
         viewModelScope.launch {
             repository.fetchCurrentConditions(lat, lon)
                 .onSuccess { _marineConditions.value = it }
+                .onFailure { android.util.Log.e("Nav", "fetchCurrentConditions failed", it) }
         }
     }
 
diff --git a/android-app/app/src/main/kotlin/org/terst/nav/ui/learn/LearnFragment.kt b/android-app/app/src/main/kotlin/org/terst/nav/ui/learn/LearnFragment.kt
index da3f020..418246c 100644
--- a/android-app/app/src/main/kotlin/org/terst/nav/ui/learn/LearnFragment.kt
+++ b/android-app/app/src/main/kotlin/org/terst/nav/ui/learn/LearnFragment.kt
@@ -1,7 +1,5 @@
 package org.terst.nav.ui.learn
 
-import android.content.Intent
-import android.net.Uri
 import android.os.Bundle
 import android.view.LayoutInflater
 import android.view.View
@@ -36,16 +34,6 @@ class LearnFragment : Fragment() {
             openDoc("docs/sailing_reference.md")
         }
 
-        // ASA / external links — open in browser
-        view.findViewById<MaterialCardView>(R.id.card_asa_courses).setOnClickListener {
-            openUrl("https://www.asa.com/courses/")
-        }
-        view.findViewById<MaterialCardView>(R.id.card_asa_online).setOnClickListener {
-            openUrl("https://www.asa.com/online-courses/")
-        }
-        view.findViewById<MaterialCardView>(R.id.card_flashcards).setOnClickListener {
-            openUrl("https://quizlet.com/subject/sailing/")
-        }
     }
 
     private fun openDoc(path: String) {
@@ -54,8 +42,4 @@ class LearnFragment : Fragment() {
             .addToBackStack(null)
             .commit()
     }
-
-    private fun openUrl(url: String) {
-        startActivity(Intent(Intent.ACTION_VIEW, Uri.parse(url)))
-    }
 }