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

Use ViewModels for each Fragment wherever possible #100

Open
basshelal opened this issue Jul 15, 2019 · 0 comments
Open

Use ViewModels for each Fragment wherever possible #100

basshelal opened this issue Jul 15, 2019 · 0 comments
Assignees
@basshelal
Labels
Backend Something regarding the backend code Idea/ Feature New idea or feature Need Refactors Something needs refactoring of some sorts

Comments

@basshelal
Copy link
Owner

This is for scalability purposes as well as overall readability. The ViewModel will actually be more of a function caller rather than a state holder but still, it's better to make the Fragment in charge of only the frontend related actions and delegate all backend actions to the ViewModel or someone else.
@basshelal basshelal added Idea/ Feature New idea or feature Need Refactors Something needs refactoring of some sorts Backend Something regarding the backend code labels Jul 15, 2019
@basshelal basshelal self-assigned this Jul 15, 2019
basshelal added a commit that referenced this issue Jul 17, 2019
@basshelal
Started with #100, separation of code to ViewModels for each Fragment
359835d
basshelal added a commit that referenced this issue Jul 17, 2019
@basshelal
Continuation on #100, separation of code to ViewModels for each Fragment
b3d4633
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@basshelal basshelal

Labels
Backend Something regarding the backend code Idea/ Feature New idea or feature Need Refactors Something needs refactoring of some sorts
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@basshelal