search

SamanthaGuillemette / SOEN390

SOEN390 Project
MIT License
5 stars 0 forks source link

Admin patient list #259

Closed AliZ786 closed 2 years ago

AliZ786 commented 2 years ago

Added actual patients from the DB on the patient's list of the admin app

smarkandu commented 2 years ago

Seems good from what I'm seeing. It's a bit redundant having both the backend stuff in both /components/patientList and then also in screens/dashboard/index. Usually for something like MVC, you'd just need to create a new view only but be able to use the same model for both. But the way we designed our code (ie. view, model and business logic mixed together), it is what it is.

I'm good with what you have. I'll let the others review and the final person to review can approve it.