Getting rid of viewmodels as a parameter from Screens

[atimadanoglu]

I think we should get rid of viewmodels as a parameter from Screen level. We should inject viewModels on Route level because when we try to preview the composables, it troubles us. Also, it creates other different problems.

From Documentation: https://developer.android.com/jetpack/compose/tooling/previews#preview-viewmodel
Warning: You shouldn't pass the ViewModel instances down to other composable functions. When you do this, you're coupling the composables with the ViewModel type, which makes them less reusable, harder to test, and unable to preview.

What do you think about that?

[waseefakhtar]

I think that's a better approach and we should go for it!

[atimadanoglu]

Can you open a task for this and assign to me?