ohcnetwork / care_fe

Care is a Digital Public Good enabling TeleICU & Decentralised Administration of Healthcare Capacity across States.
https://care.ohc.network
MIT License
204 stars 372 forks source link

Migrate to a New Model for Storing Vaccination Details of a Patient #7876

Open gigincg opened 3 months ago

gigincg commented 3 months ago

Currently CARE only supports storage of Vaccination details related to COVID. We need to generalise this and create a new model to store all vaccination details of a patient. Thereby removing these fields from the PatientRegistration Table. The Vaccination model should store comprehensive vaccination information to track a patient’s vaccination history effectively.

hrit2773 commented 3 months ago

@gigincg @rithviknishad can you assign this to me

gigincg commented 3 months ago

Hey @hrit2773 Can you outline your plan for solving this issue?

Atharva-Kanherkar commented 3 months ago

@gigincg I would love to work on the issue too. I have worked with creating models for databases, mainly in typescript and graphQL. I would love to contribute.

gigincg commented 3 months ago

@Atharva-Kanherkar The Care backend is in Django/Python. If you wanna take this up, please do explain your approach.

Since this is a core issue, we can only assign this issue post design discussions. If you would prefer to discuss over our community calls, you can find the invites in our slack

hrit2773 commented 3 months ago

Screenshot 2024-05-28 163450 @gigincg the form may look like this. Once the user enters "Yes" for the question "Is patient Vaccinated?" this will be visible and vaccine name will be a search and select form field

hrit2773 commented 3 months ago

Screenshot 2024-05-28 170621 @gigincg we can create a table kind of layout instead of just vaccine name, last vaccinated on, number of doses. This would show all the vaccination details of the patient for multiple vaccines he/she has taken.

hrit2773 commented 3 months ago

@gigincg and for backend new model Patient Vaccination can be created along with its serializers and viewsets. So backend can be done as specified along with proper validations. I can start working on both frontend and backend once this is approved

hrit2773 commented 3 months ago

@gigincg and for vaccine name choices we can add new vaccine names. Will find the names from web

hrit2773 commented 3 months ago

@gigincg any other changes from your side? Plz let me know

gigincg commented 3 months ago

@hrit2773 Assigning you the issue.

Your appriach sounds good.

Given the backend changes and migrations required, file a PR to the github.com/coronasafe/docs repo explaining the Model Changes and Migrations you will be doing before working on dev. Also please confirm if you will be making both the FE and BE PRs required.

hrit2773 commented 3 months ago

@gigincg Yes I will be doing both frontend and Backend PR's and will raise a draft PR explaining the model changes

hrit2773 commented 3 months ago

@gigincg https://github.com/coronasafe/docs/pull/27 can you just review this. I have updated both the model changes in the backend and the frontend mockups

gigincg commented 3 months ago

@hrit2773 Reviewed the docs, take a look at the notes and go ahead with the implementation (Backend and Frontend PRs)

github-actions[bot] commented 2 months ago

Hi, @coronasafe/care-frontend-maintainers, This issue has been automatically marked as stale because it has not had any recent activity.

hrit2773 commented 2 months ago

@gigincg https://github.com/coronasafe/care/pull/2275

hrit2773 commented 2 months ago

@gigincg can you plz review the frontend and backend changes. I have completed all the functionalities. Any small UI changes required? will do them immediately. cc: @rithviknishad

gigincg commented 1 week ago

@vigneshhari This will be covered the Health Details APIs right?