Senior-Project-CSE-3213 / sp_flutter_app

iOS/Android app built with Flutter using MVVM architecture
3 stars 1 forks source link

Update Architecture Diagram (move UI logic to ViewModel) #15

Closed JustinASmith closed 4 years ago

JustinASmith commented 4 years ago

The current architecture diagram needs to show that the UI logic is in fact in the ViewModel, not the View. Views are meant for only displaying data that it is given. The ViewModel has the logic that determines what that data should be.

JustinASmith commented 4 years ago

@CalebDoucet I somehow lost this draw.io file, but do you mind to just recreate this and move the label to show that the UI logic is in the ViewModel (not the view)

CalebDoucet commented 4 years ago

Updated_Arch_Diagram Sorry it took so long, but I updated the Architecture diagram. Let me know if I need to change anything about it.

JustinASmith commented 4 years ago

@CalebDoucet Thank you, but one correction to your changes. The "UI Logic" part is in the right spot on the ViewModel, but the "UI" label should remain on the view.

CalebDoucet commented 4 years ago

Updated_Arch_Diagram Fixed.