search

nkuppan / expensemanager

Enable users to store their expense & incomes and keep track of their finances.
https://expensemanager.naveenapps.com/
Apache License 2.0
109 stars 18 forks source link

Modify FilterTypeSelectionViewModel set functions #11

Closed Tonnie-Dev closed 7 months ago

Tonnie-Dev commented 8 months ago

Changed the ViewModel's set function to use MutableStateFlow.update{}. Instead of directly assigning a new value to MutableStateFlow it is better to use update{} method, which ensures thread safety and easy readability.

nkuppan commented 7 months ago

@Tonnie-Dev Appreciate your effort and please update the latest changes from the main branch.

Tonnie-Dev commented 7 months ago

@Tonnie-Dev Appreciate your effort and please update the latest changes from the main branch.

It was such a small change but I encountered this error about some failed tests. I am yet to figure it on my end. What do you think?

image