Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Introduce HomeViewModel that provides HomeUiState. #54

Open
sjaramillo10 opened this issue Mar 19, 2022 · 0 comments · May be fixed by #55
Open

Introduce HomeViewModel that provides HomeUiState. #54

sjaramillo10 opened this issue Mar 19, 2022 · 0 comments · May be fixed by #55
Labels
enhancement New feature or request
Milestone
Architecture impr...

Comments

@sjaramillo10
Copy link
Owner

The HomeFragment is currently obtaining the data it requires directly from the repository and in the Main dispatcher.

Instead of doing it this way, we should introduce a HomeViewModel that provides all the information HomeFragment requires in the form of a HomeUiState.
@sjaramillo10 sjaramillo10 added the enhancement New feature or request label Mar 19, 2022
@sjaramillo10 sjaramillo10 linked a pull request Mar 19, 2022 that will close this issue
Create HomeViewModel and make HomeFragment reactive. #55
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
Architecture improvements
Development

Successfully merging a pull request may close this issue.

Create HomeViewModel and make HomeFragment reactive. sjaramillo10/Pedometer
1 participant
@sjaramillo10