code cleanup, removed deeplink to route mapping as we can directly navigate

Author: ankitk77

app/src/main/java/pseudoankit/droid/tasky/MainActivity.kt

@@ -29,7 +29,7 @@ import kotlinx.coroutines.flow.firstOrNull
 import kotlinx.coroutines.runBlocking
 import org.koin.android.ext.android.inject
 import pseudoankit.droid.core.deeplink.TaskyDeeplink
-import pseudoankit.droid.core.logger.TaskyLogger
+import pseudoankit.droid.core.logger.logInfo
 import pseudoankit.droid.coreui.deeplink.navigateViaDeepLink
 import pseudoankit.droid.coreui.util.extension.clearStack
 import pseudoankit.droid.navigation.navgraph.mainNavGraph
@@ -107,7 +107,7 @@ internal class MainActivity : ComponentActivity() {
 
         val launcher = rememberPermissionState(permission = Manifest.permission.POST_NOTIFICATIONS)
 
-        TaskyLogger.info("Notification Permission Status", launcher.taskyStatus.name)
+        logInfo("Notification Permission Status", launcher.taskyStatus.name)
 
         LaunchedEffect(key1 = Unit) {
             launcher.launchPermissionRequest()
@@ -127,8 +127,4 @@ internal class MainActivity : ComponentActivity() {
             }
         }
     }
-
-    override fun onDestroy() {
-        super.onDestroy()
-    }
 }

core/core-ui/src/main/java/pseudoankit/droid/coreui/deeplink/DeepLinkReciever.kt

@@ -1,14 +1,16 @@
 package pseudoankit.droid.coreui.deeplink
 
+import android.net.Uri
 import androidx.navigation.NavController
-import pseudoankit.droid.core.deeplink.TaskyDeeplink
-import pseudoankit.droid.core.logger.TaskyLogger
+import pseudoankit.droid.core.logger.logError
 
-fun NavController.navigateViaDeepLink(deepLink: String) = try {
-    val internalRoute = TaskyDeeplink.mapToInternalRoute(deepLink)
-    navigate(internalRoute)
+fun NavController.navigateViaDeepLink(deepLink: String) {
+    navigateViaDeepLink(Uri.parse(deepLink))
+}
+
+fun NavController.navigateViaDeepLink(uri: Uri) = try {
+    navigate(uri)
 } catch (e: IllegalArgumentException) {
     e.printStackTrace()
-    TaskyLogger.error("deeplink [$deepLink] failed with error msg = ${e.message}")
+    logError("deeplink [${uri}] failed with error msg ${e.message}")
 }
-

core/core-ui/src/main/java/pseudoankit/droid/coreui/util/extension/ContextExtension.kt

@@ -7,9 +7,14 @@ import android.content.Intent
 import android.net.Uri
 import android.provider.Settings
 import android.widget.Toast
+import pseudoankit.droid.core.deeplink.DeepLinkUtil
 import pseudoankit.droid.core.util.TextResource
 import pseudoankit.droid.core.util.TextResource.Companion.asString
 
+fun Context.openDeeplink(deeplink: String) {
+    startActivity(DeepLinkUtil.createDeeplinkIntent(deeplink))
+}
+
 fun Context.finish() {
     (this as? Activity)?.finish()
 }

core/core/src/main/java/pseudoankit/droid/core/coroutine/CoroutineUtil.kt

@@ -6,7 +6,7 @@ import kotlinx.coroutines.Job
 import kotlinx.coroutines.launch
 import kotlin.coroutines.CoroutineContext
 
-fun launch(
+fun launchCoroutine(
     dispatcher: CoroutineContext = Dispatchers.IO,
     block: suspend CoroutineScope.() -> Unit
 ): Job {

core/core/src/main/java/pseudoankit/droid/core/deeplink/TaskyDeeplink.kt

@@ -26,17 +26,4 @@ object TaskyDeeplink {
     const val home = "${SCHEME}${Host.home}"
     const val reminder = "${SCHEME}${Host.reminder}/${Path.Reminder.action}"
     const val agendaSelection = "${SCHEME}${Host.agendaSelection}"
-
-    fun mapToInternalRoute(externalLink: String) = externalLink.run {
-        when {
-            contains(Host.home) -> "home_screen"
-            contains(Host.login) -> "login_screen"
-            contains(Host.reminder) -> {
-                replace(SCHEME + Host.reminder, "reminder_screen")
-            }
-            contains(Host.agendaSelection) -> "agenda_items_screen"
-            contains(Host.registration) -> "registration_screen"
-            else -> ""
-        }
-    }
 }

core/core/src/main/java/pseudoankit/droid/core/logger/TaskyLogger.kt

@@ -18,3 +18,7 @@ object TaskyLogger {
 fun logInfo(vararg value: Any, tag: String = "") {
     TaskyLogger.info(*value, tag)
 }
+
+fun logError(vararg value: Any, tag: String = "") {
+    TaskyLogger.error(*value, tag)
+}

core/core/src/main/java/pseudoankit/droid/core/util/extension/CollectionExtension.kt

@@ -4,7 +4,5 @@ import kotlinx.collections.immutable.ImmutableList
 import kotlinx.collections.immutable.toImmutableList
 
 inline fun <T, R> Iterable<T>.mapToImmutableList(transform: (T) -> R): ImmutableList<R> {
-    return map {
-        transform(it)
-    }.toImmutableList()
+    return map(transform).toImmutableList()
 }

core/preferences-manager/src/main/java/pseudoankit/droid/preferencesmanager/BasePreference.kt

@@ -6,6 +6,7 @@ import androidx.datastore.preferences.core.edit
 import androidx.datastore.preferences.core.emptyPreferences
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.catch
+import kotlinx.coroutines.flow.distinctUntilChanged
 import kotlinx.coroutines.flow.firstOrNull
 import kotlinx.coroutines.flow.map
 
@@ -31,5 +32,6 @@ abstract class BasePreference(
             .map { prefs ->
                 prefs[key]
             }
+            .distinctUntilChanged()
     }
 }