-
Notifications
You must be signed in to change notification settings - Fork 431
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.
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
Migrate to a New Model for Storing Vaccination Details of a Patient #7876
Comments
@gigincg @rithviknishad can you assign this to me |
Hey @hrit2773 Can you outline your plan for solving this issue? |
@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. |
@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 |
|
|
@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 |
@gigincg and for vaccine name choices we can add new vaccine names. Will find the names from web |
@gigincg any other changes from your side? Plz let me know |
@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. |
@gigincg Yes I will be doing both frontend and Backend PR's and will raise a draft PR explaining the model changes |
@gigincg ohcnetwork/docs#27 can you just review this. I have updated both the model changes in the backend and the frontend mockups |
@hrit2773 Reviewed the docs, take a look at the notes and go ahead with the implementation (Backend and Frontend PRs) |
Hi, @coronasafe/care-frontend-maintainers, This issue has been automatically marked as stale because it has not had any recent activity. |
@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 |
@vigneshhari This will be covered the Health Details APIs right? |
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.
• patient (ForeignKey to PatientRegistration)
• vaccine_name (choices from vaccineEnum)
• date_of_vaccination (DateField)
• dose_number (PositiveIntegerField, default 1)
• vaccination_center (CharField, max_length=255)
• batch_number (CharField, max_length=100)
• created_by (ForeignKey to User)
• last_edited_by (ForeignKey to User)
The text was updated successfully, but these errors were encountered: