Clean up code with Dispatchers about mynotes HOT 7 CLOSED

Can you send me the screenshot of the code itself where the Lint is giving warnings?

from mynotes.

of Course

AddTasksVM

AllTasksVM

from mynotes.

As far as the dispatcher warnings are concerned I believe they are false alarms because dispatcher.main is essentially Dispatcher.Main. Injecting Dispatchers is necessary to make Concerned ViewModel testable.
How you wish to tackle the suspending function should not be called from a different dispatcher warning?

from mynotes.

you don't need a dispatcher in viewModelScope.launch to call a suspend function, I removed the dispatchers and the app worked normally, you can check here ( S6307: Suspending functions should be main-safe )

Notice that the getTask method doesn't show the dispatcher's lint warning because it doesn't call a suspend function

from mynotes.

Try running the tests in AllTasksVMTest class and you will see they will all fail. For writing unit tests for functions which use viewModelScope it's necessary to provide a test dispatcher as Main Looper which is used by Dispatcher.Main is not available for unit tests.
One can also hack the dispatcher by using Dispatcher.setMain but injecting dispatchers is the recommended way.

from mynotes.

Ok, i will not work on dispatchers
but i found a bug when keyboar stays over the sanckbar when the user, dont select a deadline, can i work on this ?

from mynotes.

Yes this looks like a valid issue. Please raise another issue and I'll assign that to you

from mynotes.