Feature: Bookmark Sheet Integration

app/src/main/java/com/london/app/MainActivity.kt

@@ -12,15 +12,16 @@ import androidx.compose.runtime.CompositionLocalProvider
 import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.collectAsState
 import androidx.compose.runtime.getValue
-import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.runtime.setValue
+import androidx.compose.runtime.mutableStateOf
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.graphics.toArgb
 import androidx.compose.ui.platform.LocalView
 import androidx.core.view.WindowInsetsControllerCompat
 import com.london.app.navigation.NavHostGraph
 import com.london.data.local.preference.readLanguageCode
+import com.london.data.worker.MovieListSyncWorker
 import com.london.designsystem.theme.NovixTheme
 import com.london.domain.service.AppPreferencesService
 import com.london.presentation.localization.LocalizationManager
@@ -39,6 +40,9 @@ class MainActivity : ComponentActivity() {
     @Inject
     lateinit var localizationManager: LocalizationManager
 
+    @Inject
+    lateinit var workManager: androidx.work.WorkManager
+
     override fun attachBaseContext(newBase: Context) {
         val languageCode = readLanguageCode(newBase)
         val localeWrappedContext = newBase.wrapWithLocale(Locale.forLanguageTag(languageCode))
@@ -75,6 +79,8 @@ class MainActivity : ComponentActivity() {
                 }
             }
         }
+
+        MovieListSyncWorker.schedulePeriodicSync(workManager)
     }
 }
 

app/src/main/java/com/london/app/NovixApplication.kt

@@ -26,8 +26,7 @@ class NovixApplication : Application(), Configuration.Provider {
         super.onCreate()
         timberConfig()
 
-        WorkManager.initialize(this, workManagerConfiguration)
-        MovieListSyncWorker.schedulePeriodicSync(WorkManager.getInstance(applicationContext))
+        WorkManager.initialize(context = this, configuration = workManagerConfiguration)
     }
 
     private fun timberConfig() {
@@ -53,4 +52,4 @@ class NovixApplication : Application(), Configuration.Provider {
                 }
             })
     }
-}
+}

app/src/main/java/com/london/app/navigation/NavHostGraph.kt

@@ -17,7 +17,9 @@ import com.london.app.navigation.graph.mainNavGraph
 import com.london.app.navigation.graph.onboardingNavGraph
 import com.london.app.navigation.graph.splashNavGraph
 import com.london.designsystem.component.NavBar
-import com.london.designsystem.component.Scaffold
+import com.london.designsystem.snackbar.CustomSnackBarUI
+import com.london.designsystem.snackbar.ScaffoldWithSnackBar
+import com.london.designsystem.snackbar.SnackBarData
 import com.london.designsystem.theme.NovixTheme
 import com.london.presentation.navigation.LocalNavController
 import com.london.presentation.navigation.Screen.Account
@@ -32,7 +34,7 @@ fun NavHostGraph() {
     val navBackStackEntry by navController.currentBackStackEntryAsState()
     val showBottomNav = navBackStackEntry.hasRoute(Home, Search, Categories, Lists(), Account)
 
-    Scaffold(
+    ScaffoldWithSnackBar(
         containerColor = NovixTheme.colors.surface,
         bottomBar = {
             AnimatedVisibility(
@@ -48,7 +50,8 @@ fun NavHostGraph() {
                     navBackStackEntry = navBackStackEntry
                 )
             }
-        }
+        },
+        snackBar = { data: SnackBarData -> CustomSnackBarUI(data = data) }
     ) { innerPadding ->
         CompositionLocalProvider(LocalNavController provides navController) {
             NavHost(

app/src/main/java/com/london/app/navigation/graph/mainNavGraph.kt

@@ -116,7 +116,10 @@ fun NavGraphBuilder.mainNavGraph(
                 onNavigateBack = ::navigateUp
             )
         }
-        composable<Reviews> { ReviewsScreen(onNavigateBack = ::navigateUp) }
+
+        composable<Reviews> {
+            ReviewsScreen(onNavigateBack = ::navigateUp)
+        }
     }
 }
 
@@ -138,8 +141,8 @@ private fun NavGraphBuilder.homeNavGraph(navController: NavHostController) =
         composable<TopRated> {
             TopRatedScreen(
                 onNavigateBack = ::navigateUp,
-                onNaviagteToMovieDetalis = ::navigateToMovieDetails,
-                onNaviagteToTvShowDetalis = ::navigateToTvShowDetails
+                onNavigateToMovieDetails = ::navigateToMovieDetails,
+                onNavigateToTvShowDetails = ::navigateToTvShowDetails
             )
         }
     }

data/src/main/java/com/london/data/local/database/dao/customLists/SyncMetadataDao.kt

@@ -15,7 +15,10 @@ interface SyncMetadataDao {
     @Insert(onConflict = OnConflictStrategy.REPLACE)
     suspend fun insertSyncMetadata(metadata: SyncMetadataLocal)
 
+    @Query("DELETE FROM sync_metadata")
+    suspend fun clearAll()
+
     companion object {
         const val MOVIE_LISTS_SYNC_KEY = "movie_lists_sync"
     }
-}
+}

data/src/main/java/com/london/data/local/source/customLists/CustomMovieListLocalDataSourceImpl.kt

@@ -84,7 +84,7 @@ class CustomMovieListLocalDataSourceImpl @Inject constructor(
         movieListDao.updateItemCount(listId = listId, itemCount = itemCount)
 
     override suspend fun shouldRefreshCache(): Boolean {
-        val metadata = syncMetadataDao.getSyncMetadata(SYNC_KEY)
+        val metadata = syncMetadataDao.getSyncMetadata(SyncMetadataDao.MOVIE_LISTS_SYNC_KEY)
         return metadata == null ||
             !metadata.isSuccess ||
             (System.currentTimeMillis() - metadata.lastSyncTime) > CACHE_VALIDITY_MS
@@ -93,7 +93,7 @@ class CustomMovieListLocalDataSourceImpl @Inject constructor(
     override suspend fun markCacheRefreshed(success: Boolean) {
         syncMetadataDao.insertSyncMetadata(
             SyncMetadataLocal(
-                syncKey = SYNC_KEY,
+                syncKey = SyncMetadataDao.MOVIE_LISTS_SYNC_KEY,
                 lastSyncTime = System.currentTimeMillis(),
                 isSuccess = success
             )
@@ -103,10 +103,10 @@ class CustomMovieListLocalDataSourceImpl @Inject constructor(
     override suspend fun clearAllCache() {
         membershipDao.clearAll()
         movieListDao.clearAll()
+        syncMetadataDao.clearAll()
     }
 
     private companion object {
         const val CACHE_VALIDITY_MS = 30 * 60 * 1000L // 30 minutes
-        const val SYNC_KEY = SyncMetadataDao.MOVIE_LISTS_SYNC_KEY
     }
 }

data/src/main/java/com/london/data/remote/model/ApiConstants.kt

@@ -57,6 +57,6 @@ object ApiConstants {
     const val GET_LIST_DETAILS_PATH = "list/{list_id}"
     const val ADD_MOVIE_TO_LIST_PATH = "list/{list_id}/add_item"
     const val REMOVE_MOVIE_FROM_LIST_PATH = "list/{list_id}/remove_item"
-    const val GET_ACCOUNT_LISTS_PATH = "account/{account_id}/lists"
+    const val GET_ACCOUNT_LISTS_PATH = "account/0/lists"
 
 }

data/src/main/java/com/london/data/repository/authentication/AuthenticationRepositoryImpl.kt

@@ -1,6 +1,7 @@
 package com.london.data.repository.authentication
 
 import com.london.data.local.preference.AuthenticationPreferences
+import com.london.data.local.source.customLists.CustomMovieListLocalDataSource
 import com.london.data.mapper.account.toEntity
 import com.london.data.remote.model.authentication.RequestTokenResponse
 import com.london.data.remote.model.authentication.SessionResponse
@@ -13,7 +14,8 @@ import javax.inject.Inject
 class AuthenticationRepositoryImpl @Inject constructor(
     private val authenticationRemoteDataSource: AuthenticationRemoteDataSource,
     private val accountRemoteDataSource: AccountRemoteDataSource,
-    private val authenticationPreferences: AuthenticationPreferences
+    private val authenticationPreferences: AuthenticationPreferences,
+    private val customMovieListLocalDataSource: CustomMovieListLocalDataSource
 ) : AuthenticationRepository {
     override suspend fun login(username: String, password: String): Boolean {
 
@@ -51,6 +53,7 @@ class AuthenticationRepositoryImpl @Inject constructor(
         if (sessionId != null && !authenticationPreferences.isGuestMode()) {
             authenticationRemoteDataSource.deleteSession(sessionId).getOrThrow()
         }
+        customMovieListLocalDataSource.clearAllCache()
         authenticationPreferences.clearAuthentication()
         return true
     }