What will it take to pass a parcellable as an argument? <@UJBPFB3SN>

# compose

[Eric Ampire [MOD]]

What will it take to pass a parcellable as an argument? @Ian Lake

[Ian Lake]

In general, this isn't something you should consider doing: pass the ID of the item instead of the item itself. The route structure in Navigation Compose has the best analog with a restful web service profile/{id} not profile/{a whole set of fields representing a user's profile} which is essentially what passing a parcelable is doing

[Eric Ampire [MOD]]

Ok thanks

[Eric Ampire [MOD]]

There is not a method navigate with String as parameter

[arkon]

It's an extension function.

[Ian Lake]

Yep, import the extension function

[Eric Ampire [MOD]]

Doesn’t this extension come with this dependence?
androidx.navigation:navigation-composite:1.0.0-alpha01

[Eric Ampire [MOD]]

I just noticed that I had to import the method manually.

[Eric Ampire [MOD]]

Thanks

[Ian Lake]

Import should be one of the quick fixes for that error. Was that not the case for you?

[Eric Ampire [MOD]]

Exactly

[Kshitij Patil]

Does an issue have been filed regarding this behavior?

[Ian Lake]

There's no intention to change the behavior, but please do file a bug so we can properly call this out in the documentation

[Kshitij Patil]

I mean we do need to import that function manually, Android studio isn't suggesting the same

[Ian Lake]

Ah, yes you should file a bug against Studio if it isn't suggesting extension functions

[Philip Blandford]

I'm still confused.. where do I import it from?

Edit: Found it..

import androidx.navigation.compose.navigate

[Jeziel Lago]

@Ian Lake what do you think about this solution to disallow complex types as argument type? https://github.com/jeziellago/androidx/commit/fa4711838092061ca02409b998a59f12ef0b7143

[Ian Lake]

No, that's not the right approach. The problem isn't with having arguments with complex types - that's actually totally fine (and you could absolutely use an EnumType with a defaultValue to build an argument based listener ). It is easy to conflate the list of arguments in your route with the total list of arguments, but those are purposefully separate things

[Jeziel Lago]

Interesting! Thanks!!

[Florian]

@Ian Lake When we send a result back through the previous back stack entries' SavedStateHandle, putting a Parcelable object in there is no problem. It's unclear to me if it's fine to use Parcelable there or if we should avoid that as well. Because why send Parcelables in one direction but not the other?

[Ian Lake]

The first thing you should do is define your single source of truth. An observable repository is an example of a source of truth - any destination can read that data and get the same answer back. Any source can update that data and have it automatically flow back to everyone using that source of data. Arguments and results should not be a source of truth for your app's data

[Florian]

@Ian Lake. I see, thank you. The Parcelable I'm sending back is not an entity. It's a sealed result class. Is it fine to put that in the SavedStateHandle of the previousBackstackEntry?

[Ian Lake]

Nothing is going to stop you 👍

[Florian]

Alright, thank you very much!