Status Update
No update yet.
Comments
wi...@gmail.com
Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <
Link:
Add non-reified NavGraphBuilder nested graph DSL
Expand for full commit details
Add non-reified NavGraphBuilder nested graph DSL
Test: ./gradlew navigation:navigation-common:cC
Bug: 382094941
Relnote: "Added non-reified NavGraphBuilder.navigation extention functions to build a nested graph with route from KClass<T>"
Change-Id: Ibf24453e66d69c00d4ffb3abe4dc5a1f3782ad09
Files:
- M
navigation/navigation-common/api/current.txt - M
navigation/navigation-common/api/restricted_current.txt - M
navigation/navigation-common/src/androidTest/java/androidx/navigation/NavGraphBuilderTest.kt - M
navigation/navigation-common/src/main/java/androidx/navigation/NavGraphBuilder.kt
Hash: 94c09a993d68e5c57172c3e3bc418e35ec57f2fc
Date: Tue Dec 03 16:52:10 2024
jb...@google.com
cl...@google.com
ap...@google.com
Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <
Link:
Add non-reified NavGraph.findNode safe args api
Expand for full commit details
Add non-reified NavGraph.findNode safe args api
Test: ./gradlew navigation:navigation-common:cC
Bug: 382094941
Relnote: "Added non-reified NavGraph.findNode API to find node with KClass<*>"
Change-Id: I54696ee5ce64305386de64654c6ce15f3a1fd741
Files:
- M
navigation/navigation-common/api/current.txt - M
navigation/navigation-common/api/restricted_current.txt - M
navigation/navigation-common/src/androidTest/java/androidx/navigation/NavGraphTest.kt - M
navigation/navigation-common/src/main/java/androidx/navigation/NavGraph.kt
Hash: 3c80f14b77e8f2f84f961b24de924439fc009ad7
Date: Tue Dec 03 13:15:22 2024
cl...@google.com
Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <
Link:
Add non-reified NavDeepLink.Builder.setUriPattern safe args api
Expand for full commit details
Add non-reified NavDeepLink.Builder.setUriPattern safe args api
Test: existing tests
Bug: 382094941
Relnote: "Added non-reified NavDeepLink.Builder.setUriPattern API to add set deepLink Uri pattern with KClass<*>"
Change-Id: Ia47be2a0ac9e2b8ef5def4aa808b939e15259d2a
Files:
- M
navigation/navigation-common/api/current.txt - M
navigation/navigation-common/api/restricted_current.txt - M
navigation/navigation-common/src/main/java/androidx/navigation/NavDeepLink.kt
Hash: 92ab76b0551be25fab92d2884a8c0c06a521af41
Date: Tue Dec 03 13:06:15 2024
pr...@google.com
Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <
Link:
Add non-reified NavDestinationBuilder.deepLink safe args api
Expand for full commit details
Add non-reified NavDestinationBuilder.deepLink safe args api
Test: ./gradlew navigation:navigation-common:cC
Bug: 382094941
Relnote: "Added non-reified NavDestinationBuilder.deepLink API to add a deeplink with KClass<*>"
Change-Id: I93281ed88fe3be9e6c68a2d5f16d9e0f335c999d
Files:
- M
navigation/navigation-common/api/current.txt - M
navigation/navigation-common/api/restricted_current.txt - M
navigation/navigation-common/src/androidTest/java/androidx/navigation/NavDestinationBuilderTest.kt - M
navigation/navigation-common/src/main/java/androidx/navigation/NavDestinationBuilder.kt
Hash: 3de22e912b5752790370c40acdc03de4d0985933
Date: Tue Dec 03 12:57:11 2024
Description
Version used: 2.7.7
Devices/Android versions reproduced on: Pixel 3, Pixel 7 Pro
We recently experienced an app crash when switching our NavHost's startDestination. After some investigation, it seems like this is triggered only after having previously navigated between two routes, where the second route is nested inside of a nav graph whose route has the same name as a sibling of the starting route (causing them to have the same ID).
Fortunately, this means the crash can be avoided by avoiding those duplicates, but it was a bit tricky to find the cause of the NullPointerException, and it would be nice if it failed more gracefully than a NullPointerException with the possibility of crashing the app entirely.
I have attached a sample app which reproduces this crash.