Skip to content

Moved coroutine usage in the ViewModels#167

Open
angelomarzocchi wants to merge 1 commit intogoogle-developer-training:mainfrom
angelomarzocchi:main
Open

Moved coroutine usage in the ViewModels#167
angelomarzocchi wants to merge 1 commit intogoogle-developer-training:mainfrom
angelomarzocchi:main

Conversation

@angelomarzocchi
Copy link

Updated Gradle clean task to make it compatible with new Gradle version.
I also moved couroutine usage in the ViewModel for 2 reasons:

  1. it better follows the single responability principle: I don't see why calling a coroutine is a responsability of the UI.
  2. The problem of the update cancelled when recomposition is being performed is not an issue anymore :)

Updated Gradle clean task to make it compatible with new Gradle version.
Moved coroutine usage in the ViewModel, where it belongs.
@angelomarzocchi angelomarzocchi changed the title Updated dependencies. Moved coroutine usage in the ViewModels Dec 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant