Skip to content

Commit

Permalink
chore: remove deprecated accompanist-navigation dependency (#2568)
Browse files Browse the repository at this point in the history
  • Loading branch information
ohassine committed Jan 9, 2024
1 parent c11b6b5 commit 2124283
Show file tree
Hide file tree
Showing 7 changed files with 58 additions and 57 deletions.
1 change: 0 additions & 1 deletion app/build.gradle.kts
Expand Up @@ -118,7 +118,6 @@ dependencies {
// Accompanist
implementation(libs.accompanist.systemUI)
implementation(libs.accompanist.placeholder)
implementation(libs.accompanist.navAnimation)

implementation(libs.androidx.paging3)
implementation(libs.androidx.paging3Compose)
Expand Down
Expand Up @@ -17,22 +17,23 @@
*/
package com.wire.android.navigation

import androidx.compose.animation.ExperimentalAnimationApi
import androidx.compose.runtime.Composable
import androidx.navigation.NavDestination
import androidx.navigation.compose.rememberNavController
import com.datadog.android.compose.ExperimentalTrackingApi
import com.datadog.android.compose.NavigationViewTrackingEffect
import com.datadog.android.rum.tracking.AcceptAllNavDestinations
import com.google.accompanist.navigation.animation.rememberAnimatedNavController

@OptIn(ExperimentalAnimationApi::class, ExperimentalTrackingApi::class)
@OptIn(ExperimentalTrackingApi::class)
@Composable
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) = rememberAnimatedNavController().apply {
NavigationViewTrackingEffect(
navController = this,
trackArguments = true,
destinationPredicate = object : AcceptAllNavDestinations() {
override fun getViewName(component: NavDestination): String? = component.route?.let { nameFromRoute(it) }
}
)
}
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) =
rememberNavController().apply {
NavigationViewTrackingEffect(
navController = this,
trackArguments = true,
destinationPredicate = object : AcceptAllNavDestinations() {
override fun getViewName(component: NavDestination): String? =
component.route?.let { nameFromRoute(it) }
}
)
}
@@ -1,21 +1,22 @@
package com.wire.android.navigation

import androidx.compose.animation.ExperimentalAnimationApi
import androidx.compose.runtime.Composable
import androidx.navigation.NavDestination
import androidx.navigation.compose.rememberNavController
import com.datadog.android.compose.ExperimentalTrackingApi
import com.datadog.android.compose.NavigationViewTrackingEffect
import com.datadog.android.rum.tracking.AcceptAllNavDestinations
import com.google.accompanist.navigation.animation.rememberAnimatedNavController

@OptIn(ExperimentalAnimationApi::class, ExperimentalTrackingApi::class)
@OptIn(ExperimentalTrackingApi::class)
@Composable
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) = rememberAnimatedNavController().apply {
NavigationViewTrackingEffect(
navController = this,
trackArguments = true,
destinationPredicate = object : AcceptAllNavDestinations() {
override fun getViewName(component: NavDestination): String? = component.route?.let { nameFromRoute(it) }
}
)
}
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) =
rememberNavController().apply {
NavigationViewTrackingEffect(
navController = this,
trackArguments = true,
destinationPredicate = object : AcceptAllNavDestinations() {
override fun getViewName(component: NavDestination): String? =
component.route?.let { nameFromRoute(it) }
}
)
}
@@ -1,21 +1,22 @@
package com.wire.android.navigation

import androidx.compose.animation.ExperimentalAnimationApi
import androidx.compose.runtime.Composable
import androidx.navigation.NavDestination
import com.datadog.android.compose.ExperimentalTrackingApi
import com.datadog.android.compose.NavigationViewTrackingEffect
import com.datadog.android.rum.tracking.AcceptAllNavDestinations
import com.google.accompanist.navigation.animation.rememberAnimatedNavController
import androidx.navigation.compose.rememberNavController

@OptIn(ExperimentalAnimationApi::class, ExperimentalTrackingApi::class)
@OptIn(ExperimentalTrackingApi::class)
@Composable
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) = rememberAnimatedNavController().apply {
NavigationViewTrackingEffect(
navController = this,
trackArguments = true,
destinationPredicate = object : AcceptAllNavDestinations() {
override fun getViewName(component: NavDestination): String? = component.route?.let { nameFromRoute(it) }
}
)
}
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) =
rememberNavController().apply {
NavigationViewTrackingEffect(
navController = this,
trackArguments = true,
destinationPredicate = object : AcceptAllNavDestinations() {
override fun getViewName(component: NavDestination): String? =
component.route?.let { nameFromRoute(it) }
}
)
}
@@ -1,9 +1,8 @@
package com.wire.android.navigation

import androidx.compose.animation.ExperimentalAnimationApi
import androidx.compose.runtime.Composable
import com.google.accompanist.navigation.animation.rememberAnimatedNavController
import androidx.navigation.compose.rememberNavController

@OptIn(ExperimentalAnimationApi::class)
@Composable
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) = rememberAnimatedNavController()
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) =
rememberNavController()
@@ -1,21 +1,22 @@
package com.wire.android.navigation

import androidx.compose.animation.ExperimentalAnimationApi
import androidx.compose.runtime.Composable
import androidx.navigation.NavDestination
import com.datadog.android.compose.ExperimentalTrackingApi
import com.datadog.android.compose.NavigationViewTrackingEffect
import com.datadog.android.rum.tracking.AcceptAllNavDestinations
import com.google.accompanist.navigation.animation.rememberAnimatedNavController
import androidx.navigation.compose.rememberNavController

@OptIn(ExperimentalAnimationApi::class, ExperimentalTrackingApi::class)
@OptIn(ExperimentalTrackingApi::class)
@Composable
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) = rememberAnimatedNavController().apply {
NavigationViewTrackingEffect(
navController = this,
trackArguments = true,
destinationPredicate = object : AcceptAllNavDestinations() {
override fun getViewName(component: NavDestination): String? = component.route?.let { nameFromRoute(it) }
}
)
}
fun rememberTrackingAnimatedNavController(nameFromRoute: (String) -> String?) =
rememberNavController().apply {
NavigationViewTrackingEffect(
navController = this,
trackArguments = true,
destinationPredicate = object : AcceptAllNavDestinations() {
override fun getViewName(component: NavDestination): String? =
component.route?.let { nameFromRoute(it) }
}
)
}
7 changes: 3 additions & 4 deletions gradle/libs.versions.toml
Expand Up @@ -37,11 +37,11 @@ androidx-browser = "1.5.0"
androidx-biometric = "1.1.0"

# Compose
composeBom = "2023.10.00" # TODO check if in new version [anchoredDraggable] is available
compose-activity = "1.8.0"
composeBom = "2023.10.01" # TODO check if in new version [anchoredDraggable] is available
compose-activity = "1.8.2"
compose-compiler = "1.5.2"
compose-constraint = "1.0.1"
compose-navigation = "2.7.3" # adjusted to work with compose-destinations "1.9.54"
compose-navigation = "2.7.6" # adjusted to work with compose-destinations "1.9.54"
compose-destinations = "1.9.54"

# Hilt
Expand Down Expand Up @@ -188,7 +188,6 @@ compose-destinations-ksp = { module = "io.github.raamcosta.compose-destinations:
# Accompanist
accompanist-systemUI = { module = "com.google.accompanist:accompanist-systemuicontroller", version.ref = "accompanist" }
accompanist-placeholder = { module = "com.google.accompanist:accompanist-placeholder", version.ref = "accompanist" }
accompanist-navAnimation = { module = "com.google.accompanist:accompanist-navigation-animation", version.ref = "accompanist" }

# Image Loading
coil-core = { module = "io.coil-kt:coil", version.ref = "coil" }
Expand Down

0 comments on commit 2124283

Please sign in to comment.