Merge pull request #133 from CrisisCleanup/improve-data-caching

Improve data caching

Author: hueachilles

app-sandbox/build.gradle.kts

@@ -63,6 +63,7 @@ dependencies {
     implementation(libs.androidx.profileinstaller)
 
     implementation(libs.coil.kt)
+    implementation(libs.coil.kt.compose)
 
     implementation(libs.kotlinx.coroutines.playservices)
     implementation(libs.playservices.maps)

app-sandbox/src/main/java/com/crisiscleanup/sandbox/SandboxApp.kt

@@ -24,8 +24,9 @@ import androidx.navigation.NavController
 import com.crisiscleanup.core.designsystem.component.CrisisCleanupBackground
 import com.crisiscleanup.core.designsystem.component.CrisisCleanupTextButton
 import com.crisiscleanup.core.designsystem.theme.listItemSpacedBy
-import com.crisiscleanup.sandbox.navigation.MULTI_IMAGE_ROUTE
+import com.crisiscleanup.sandbox.navigation.ASYNC_IMAGE_ROUTE
 import com.crisiscleanup.sandbox.navigation.SandboxNavHost
+import com.crisiscleanup.sandbox.navigation.navigateToAsyncImage
 import com.crisiscleanup.sandbox.navigation.navigateToBottomNav
 import com.crisiscleanup.sandbox.navigation.navigateToCheckboxes
 import com.crisiscleanup.sandbox.navigation.navigateToChips
@@ -65,7 +66,7 @@ fun SandboxApp(
                     SandboxNavHost(
                         appState.navController,
                         appState::onBack,
-                        MULTI_IMAGE_ROUTE,
+                        ASYNC_IMAGE_ROUTE,
                     )
                 }
             }
@@ -98,6 +99,9 @@ fun RootRoute(navController: NavController) {
             CrisisCleanupTextButton(text = "Images") {
                 navController.navigateToMultiImage()
             }
+            CrisisCleanupTextButton(text = "Async image") {
+                navController.navigateToAsyncImage()
+            }
         }
     }
 }

app-sandbox/src/main/java/com/crisiscleanup/sandbox/SandboxApplication.kt

@@ -134,8 +134,6 @@ class AppSyncer @Inject constructor() : SyncPuller, SyncPusher {
 
     override suspend fun syncPushWorksitesAsync() = CompletableDeferred(SyncResult.NotAttempted(""))
 
-    override fun stopPushWorksites() {}
-
     override suspend fun syncPushMedia() = SyncResult.NotAttempted("")
 
     override suspend fun syncPushWorksites() = SyncResult.NotAttempted("")

app-sandbox/src/main/java/com/crisiscleanup/sandbox/navigation/SandboxNavigation.kt

@@ -6,6 +6,7 @@ import androidx.navigation.NavHostController
 import androidx.navigation.compose.NavHost
 import androidx.navigation.compose.composable
 import com.crisiscleanup.sandbox.RootRoute
+import com.crisiscleanup.sandbox.ui.AsyncImageView
 import com.crisiscleanup.sandbox.ui.BottomNavRoute
 import com.crisiscleanup.sandbox.ui.CheckboxesRoute
 import com.crisiscleanup.sandbox.ui.ChipsRoute
@@ -18,6 +19,7 @@ private const val CHIPS_ROUTE = "chips"
 private const val BOTTOM_NAV_ROUTE = "bottom-nav"
 const val SINGLE_IMAGE_ROUTE = "single-image"
 const val MULTI_IMAGE_ROUTE = "multi-image"
+const val ASYNC_IMAGE_ROUTE = "async-image"
 
 fun NavController.navigateToBottomNav() {
     this.navigate(BOTTOM_NAV_ROUTE)
@@ -39,6 +41,10 @@ fun NavController.navigateToMultiImage() {
     this.navigate(MULTI_IMAGE_ROUTE)
 }
 
+fun NavController.navigateToAsyncImage() {
+    this.navigate(ASYNC_IMAGE_ROUTE)
+}
+
 @Composable
 fun SandboxNavHost(
     navController: NavHostController,
@@ -72,5 +78,9 @@ fun SandboxNavHost(
         composable(MULTI_IMAGE_ROUTE) {
             MultiImageRoute(onBack)
         }
+
+        composable(ASYNC_IMAGE_ROUTE) {
+            AsyncImageView()
+        }
     }
 }

app-sandbox/src/main/java/com/crisiscleanup/sandbox/ui/AsyncImageView.kt

@@ -0,0 +1,51 @@
+package com.crisiscleanup.sandbox.ui
+
+import android.util.Log
+import androidx.compose.foundation.layout.fillMaxSize
+import androidx.compose.foundation.layout.sizeIn
+import androidx.compose.foundation.shape.RoundedCornerShape
+import androidx.compose.runtime.Composable
+import androidx.compose.ui.Modifier
+import androidx.compose.ui.draw.clip
+import androidx.compose.ui.graphics.vector.rememberVectorPainter
+import androidx.compose.ui.layout.ContentScale
+import androidx.compose.ui.platform.LocalContext
+import androidx.compose.ui.res.painterResource
+import androidx.compose.ui.unit.dp
+import coil.compose.AsyncImage
+import coil.request.ImageRequest
+import com.crisiscleanup.core.common.R
+import com.crisiscleanup.core.designsystem.icon.CrisisCleanupIcons
+
+@Composable
+fun AsyncImageView() {
+    val imageUrl = ""
+    AsyncImage(
+        model = ImageRequest.Builder(LocalContext.current)
+            .data(imageUrl)
+            .setHeader("User-Agent", "Mozilla/5.0")
+            .build(),
+        modifier = Modifier
+            .sizeIn(minWidth = 96.dp)
+            .fillMaxSize()
+            .clip(RoundedCornerShape(8.dp)),
+        placeholder = painterResource(R.drawable.cc_grayscale_pin),
+        error = rememberVectorPainter(CrisisCleanupIcons.SyncProblem),
+        fallback = rememberVectorPainter(CrisisCleanupIcons.Image),
+        contentDescription = null,
+        contentScale = ContentScale.Fit,
+        onLoading = { loading ->
+            Log.i("async-image", "Loading image $loading")
+        },
+        onSuccess = { success ->
+            Log.i("async-image", "Successfully loaded ${success.result}")
+        },
+        onError = { e ->
+            Log.e(
+                "async-image",
+                "Error loading image ${e.result} for $imageUrl",
+                e.result.throwable,
+            )
+        },
+    )
+}

app/build.gradle.kts

@@ -14,7 +14,7 @@ plugins {
 
 android {
     defaultConfig {
-        val buildVersion = 247
+        val buildVersion = 250
         applicationId = "com.crisiscleanup"
         versionCode = buildVersion
         versionName = "0.9.${buildVersion - 168}"

app/src/main/java/com/crisiscleanup/MainActivityViewModel.kt

@@ -198,6 +198,7 @@ class MainActivityViewModel @Inject constructor(
 
         incidentSelector.incidentId
             .filter { it != EmptyIncident.id }
+            .distinctUntilChanged()
             .onEach {
                 sync(
                     forcePullIncidents = false,

core/app-component/src/main/kotlin/com/crisiscleanup/core/appcomponent/AppTopBarDataProvider.kt

@@ -43,6 +43,8 @@ class AppTopBarDataProvider(
 
     val showHeaderLoading = incidentCacheRepository.isSyncingActiveIncident
 
+    val enableIncidentSelect = incidentsRepository.isFirstLoad.map(Boolean::not)
+
     val screenTitle = incidentSelector.incident
         .map { it.shortName.ifBlank { translator(screenTitleKey) } }
         .stateIn(

core/app-component/src/main/kotlin/com/crisiscleanup/core/appcomponent/ui/AppTopBar.kt

@@ -28,6 +28,7 @@ fun AppTopBar(
     val isHeaderLoading by dataProvider.showHeaderLoading.collectAsState(false)
 
     val disasterIconResId by dataProvider.disasterIconResId.collectAsStateWithLifecycle()
+    val enableIncidentSelect by dataProvider.enableIncidentSelect.collectAsState(false)
 
     val isAccountExpired by dataProvider.isAccountExpired.collectAsStateWithLifecycle()
     val profilePictureUri by dataProvider.profilePictureUri.collectAsStateWithLifecycle()
@@ -42,6 +43,7 @@ fun AppTopBar(
         isAccountExpired = isAccountExpired,
         openAuthentication = openAuthentication,
         disasterIconResId = disasterIconResId,
+        enableIncidentSelect = enableIncidentSelect,
         onOpenIncidents = onOpenIncidents,
     )
 }
@@ -60,6 +62,7 @@ internal fun AppTopBar(
     isAccountExpired: Boolean = false,
     openAuthentication: () -> Unit = {},
     @DrawableRes disasterIconResId: Int = 0,
+    enableIncidentSelect: Boolean = false,
     onOpenIncidents: (() -> Unit)? = null,
 ) {
     val t = LocalAppTranslator.current
@@ -86,6 +89,7 @@ internal fun AppTopBar(
                     title = title,
                     contentDescription = t("nav.change_incident"),
                     isLoading = isAppHeaderLoading,
+                    enabled = enableIncidentSelect,
                 )
             }
         },

core/common/src/main/java/com/crisiscleanup/core/common/sync/Syncer.kt

@@ -54,7 +54,6 @@ interface SyncPuller {
 interface SyncPusher {
     fun appPushWorksite(worksiteId: Long, scheduleMediaSync: Boolean = false)
     suspend fun syncPushWorksitesAsync(): Deferred<SyncResult>
-    fun stopPushWorksites()
     suspend fun syncPushMedia(): SyncResult
     suspend fun syncPushWorksites(): SyncResult