Hello <@UPWE6PJCE> <@U9D52BJF2>! Do you think it would be useful to have `ContainerHostExtensions.k...

# orbit-mvi

[Guilherme Delgado]

Hello @Mikolaj Leszczynski @appmattus!
Do you think it would be useful to have ContainerHostExtensions.kt using state: Lifecycle.State = Lifecycle.State.STARTED as a parameter? 🤔 This way if for some reason we need to change it, we could.

[appmattus]

Yes no issues from me

[Guilherme Delgado]

shall i create a PR or you handle it on your side? I will create it 😊

[Guilherme Delgado]

https://github.com/orbit-mvi/orbit-mvi/pull/120

[Guilherme Delgado]

done 😇

[Mikolaj Leszczynski]

Thanks!

Worth also updating org.orbitmvi.orbit.viewmodel.ContainerHostExtensionsKt#observe for consistency.

[Guilherme Delgado]

on it 😉

[Guilherme Delgado]

(thanks for pointing out, missed that one 😅 )

[Mikolaj Leszczynski]

hah no worries

[Guilherme Delgado]

done 🙂

[appmattus]

PR merged