fix(waypoints): migrationCompleteFlow never set after legacy storage migration

growse · growse · commit 291632e08717 · 2026-04-20T10:02:09.000+01:00
Fixes #2224
diff --git a/project/app/src/main/java/org/owntracks/android/data/waypoints/RoomWaypointsRepo.kt b/project/app/src/main/java/org/owntracks/android/data/waypoints/RoomWaypointsRepo.kt
@@ -133,6 +133,9 @@ constructor(
     val handler = CoroutineExceptionHandler { _, exception ->
       Timber.e(exception, "Error migrating waypoints")
     }
-    scope.launch(ioDispatcher + handler) { migrateFromLegacyStorage() }
+    scope.launch(ioDispatcher + handler) {
+      migrateFromLegacyStorage()
+      _migrationCompleteFlow.value = true
+    }
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -133,6 +133,9 @@ constructor(`
`133`	`133`	`val handler = CoroutineExceptionHandler { _, exception ->`
`134`	`134`	`Timber.e(exception, "Error migrating waypoints")`
`135`	`135`	`}`
`136`		`- scope.launch(ioDispatcher + handler) { migrateFromLegacyStorage() }`
	`136`	`+ scope.launch(ioDispatcher + handler) {`
	`137`	`+ migrateFromLegacyStorage()`
	`138`	`+ _migrationCompleteFlow.value = true`
	`139`	`+ }`
`137`	`140`	`}`
`138`	`141`	`}`