Merge pull request android#341 from yrezgui/jetcaster-alpha10

yrezgui · web-flow · commit 61a47f5bad17 · 2021-01-13T17:45:03.000Z
[Jetcaster] Bump to alpha10 snapshot
diff --git a/Jetcaster/app/build.gradle b/Jetcaster/app/build.gradle
@@ -97,7 +97,9 @@ dependencies {
     implementation Libs.Kotlin.stdlib
     implementation Libs.Coroutines.android
 
-    implementation Libs.AndroidX.coreKtx
+    // Accompanist.insets must depend on the SNAPSHOT version of core:1.5.0
+    implementation(Libs.AndroidX.core) { force true }
+    implementation(Libs.AndroidX.coreKtx) { force true }
     implementation Libs.AndroidX.palette
 
     implementation Libs.AndroidX.Lifecycle.viewmodel
diff --git a/Jetcaster/app/src/main/AndroidManifest.xml b/Jetcaster/app/src/main/AndroidManifest.xml
@@ -17,6 +17,8 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="com.example.jetcaster">
 
+    <!-- Uses ACCESS_NETWORK_STATE to check if the device is connected to internet or not -->
+    <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
     <!-- Uses INTERNET to fetch RSS feed + images -->
     <uses-permission android:name="android.permission.INTERNET" />
 
diff --git a/Jetcaster/app/src/main/java/com/example/jetcaster/data/PodcastsRepository.kt b/Jetcaster/app/src/main/java/com/example/jetcaster/data/PodcastsRepository.kt
@@ -16,6 +16,7 @@
 
 package com.example.jetcaster.data
 
+import android.util.Log
 import com.example.jetcaster.data.room.TransactionRunner
 import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.CoroutineScope
@@ -43,22 +44,26 @@ class PodcastsRepository(
             refreshingJob?.join()
         } else if (force || podcastStore.isEmpty()) {
             refreshingJob = scope.launch {
-                // Now fetch the podcasts, and add each to each store
-                podcastsFetcher(SampleFeeds).collect { (podcast, episodes, categories) ->
-                    transactionRunner {
-                        podcastStore.addPodcast(podcast)
-                        episodeStore.addEpisodes(episodes)
+                try {
+                    // Now fetch the podcasts, and add each to each store
+                    podcastsFetcher(SampleFeeds).collect { (podcast, episodes, categories) ->
+                        transactionRunner {
+                            podcastStore.addPodcast(podcast)
+                            episodeStore.addEpisodes(episodes)
 
-                        categories.forEach { category ->
-                            // First insert the category
-                            val categoryId = categoryStore.addCategory(category)
-                            // Now we can add the podcast to the category
-                            categoryStore.addPodcastToCategory(
-                                podcastUri = podcast.uri,
-                                categoryId = categoryId
-                            )
+                            categories.forEach { category ->
+                                // First insert the category
+                                val categoryId = categoryStore.addCategory(category)
+                                // Now we can add the podcast to the category
+                                categoryStore.addPodcastToCategory(
+                                    podcastUri = podcast.uri,
+                                    categoryId = categoryId
+                                )
+                            }
                         }
                     }
+                } catch (e: Throwable) {
+                    Log.d("PodcastsRepository", "podcastsFetcher(SampleFeeds).collect error: $e")
                 }
             }
         }
diff --git a/Jetcaster/app/src/main/java/com/example/jetcaster/ui/JetcasterApp.kt b/Jetcaster/app/src/main/java/com/example/jetcaster/ui/JetcasterApp.kt
@@ -16,11 +16,52 @@
 
 package com.example.jetcaster.ui
 
+import android.content.Context
+import android.net.ConnectivityManager
+import androidx.compose.material.AlertDialog
+import androidx.compose.material.Text
+import androidx.compose.material.TextButton
 import androidx.compose.runtime.Composable
+import androidx.compose.runtime.getValue
+import androidx.compose.runtime.mutableStateOf
+import androidx.compose.runtime.remember
+import androidx.compose.runtime.setValue
+import androidx.compose.ui.platform.AmbientContext
+import androidx.compose.ui.res.stringResource
+import com.example.jetcaster.R
 import com.example.jetcaster.ui.home.Home
 
 @Composable
 fun JetcasterApp() {
+    val context = AmbientContext.current
+    var isOnline by remember { mutableStateOf(checkIfOnline(context)) }
+
     // TODO: add some navigation
-    Home()
+    if (isOnline) {
+        Home()
+    } else {
+        OfflineDialog { isOnline = checkIfOnline(context) }
+    }
+}
+
+// TODO: Use a better way to check internet connection
+@Suppress("DEPRECATION")
+private fun checkIfOnline(context: Context): Boolean {
+    val cm = context.getSystemService(Context.CONNECTIVITY_SERVICE) as ConnectivityManager
+    val activeNetwork = cm.activeNetworkInfo
+    return activeNetwork?.isConnectedOrConnecting == true
+}
+
+@Composable
+fun OfflineDialog(onRetry: () -> Unit) {
+    AlertDialog(
+        onDismissRequest = {},
+        title = { Text(text = stringResource(R.string.connection_error_title)) },
+        text = { Text(text = stringResource(R.string.connection_error_message)) },
+        confirmButton = {
+            TextButton(onClick = onRetry) {
+                Text(stringResource(R.string.retry_label))
+            }
+        }
+    )
 }
diff --git a/Jetcaster/app/src/main/java/com/example/jetcaster/util/Buttons.kt b/Jetcaster/app/src/main/java/com/example/jetcaster/util/Buttons.kt
@@ -16,7 +16,8 @@
 
 package com.example.jetcaster.util
 
-import androidx.compose.animation.animate
+import androidx.compose.animation.animateAsState
+import androidx.compose.animation.core.animateAsState
 import androidx.compose.foundation.background
 import androidx.compose.foundation.layout.padding
 import androidx.compose.material.AmbientContentColor
@@ -49,24 +50,24 @@ fun ToggleFollowPodcastIconButton(
                 isFollowed -> Icons.Default.Check
                 else -> Icons.Default.Add
             },
-            tint = animate(
+            tint = animateAsState(
                 when {
                     isFollowed -> AmbientContentColor.current
                     else -> Color.Black.copy(alpha = ContentAlpha.high)
                 }
-            ),
+            ).value,
             modifier = Modifier
                 .shadow(
-                    elevation = animate(if (isFollowed) 0.dp else 1.dp),
+                    elevation = animateAsState(if (isFollowed) 0.dp else 1.dp).value,
                     shape = MaterialTheme.shapes.small
                 )
                 .background(
-                    color = animate(
+                    color = animateAsState(
                         when {
                             isFollowed -> MaterialTheme.colors.surface.copy(0.38f)
                             else -> Color.White
                         }
-                    ),
+                    ).value,
                     shape = MaterialTheme.shapes.small
                 )
                 .padding(4.dp)
diff --git a/Jetcaster/app/src/main/java/com/example/jetcaster/util/DynamicTheming.kt b/Jetcaster/app/src/main/java/com/example/jetcaster/util/DynamicTheming.kt
@@ -18,7 +18,7 @@ package com.example.jetcaster.util
 
 import android.content.Context
 import androidx.collection.LruCache
-import androidx.compose.animation.animate
+import androidx.compose.animation.animateAsState
 import androidx.compose.animation.core.Spring
 import androidx.compose.animation.core.spring
 import androidx.compose.material.MaterialTheme
@@ -61,8 +61,14 @@ fun DynamicThemePrimaryColorsFromImage(
     content: @Composable () -> Unit
 ) {
     val colors = MaterialTheme.colors.copy(
-        primary = animate(dominantColorState.color, spring(stiffness = Spring.StiffnessLow)),
-        onPrimary = animate(dominantColorState.onColor, spring(stiffness = Spring.StiffnessLow))
+        primary = animateAsState(
+            dominantColorState.color,
+            spring(stiffness = Spring.StiffnessLow)
+        ).value,
+        onPrimary = animateAsState(
+            dominantColorState.onColor,
+            spring(stiffness = Spring.StiffnessLow)
+        ).value
     )
     MaterialTheme(colors = colors, content = content)
 }
diff --git a/Jetcaster/app/src/main/res/values/strings.xml b/Jetcaster/app/src/main/res/values/strings.xml
@@ -16,6 +16,11 @@
   -->
 <resources>
     <string name="app_name">Jetcaster</string>
+
+    <string name="connection_error_title">Connection error</string>
+    <string name="connection_error_message">Unable to fetch podcasts feeds.\nCheck your internet connection and try again.</string>
+    <string name="retry_label">Retry</string>
+    
     <string name="your_podcasts">Your podcasts</string>
     <string name="latest_episodes">Latest episodes</string>
 
diff --git a/Jetcaster/buildSrc/src/main/java/com/example/jetcaster/buildsrc/dependencies.kt b/Jetcaster/buildSrc/src/main/java/com/example/jetcaster/buildsrc/dependencies.kt
@@ -21,15 +21,15 @@ object Versions {
 }
 
 object Libs {
-    const val androidGradlePlugin = "com.android.tools.build:gradle:7.0.0-alpha03"
+    const val androidGradlePlugin = "com.android.tools.build:gradle:7.0.0-alpha04"
     const val jdkDesugar = "com.android.tools:desugar_jdk_libs:1.0.9"
 
     const val junit = "junit:junit:4.13"
 
     const val material = "com.google.android.material:material:1.1.0"
 
     object Accompanist {
-        private const val version = "0.4.1"
+        private const val version = "0.4.1.compose-7067732-SNAPSHOT"
         const val coil = "dev.chrisbanes.accompanist:accompanist-coil:$version"
         const val insets = "dev.chrisbanes.accompanist:accompanist-insets:$version"
     }
@@ -58,12 +58,12 @@ object Libs {
         const val appcompat = "androidx.appcompat:appcompat:1.2.0"
         const val palette = "androidx.palette:palette:1.0.0"
 
-        const val core = "androidx.core:core:1.5.0-alpha04"
-        const val coreKtx = "androidx.core:core-ktx:1.5.0-alpha04"
+        const val core = "androidx.core:core:1.5.0-SNAPSHOT"
+        const val coreKtx = "androidx.core:core-ktx:1.5.0-SNAPSHOT"
 
         object Compose {
-            private const val snapshot = ""
-            private const val version = "1.0.0-alpha09"
+            private const val snapshot = "7067732"
+            private const val version = "1.0.0-SNAPSHOT"
 
             @get:JvmStatic
             val snapshotUrl: String
