Added more fields to Consultation Form #979
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
Waiting for FE before merging |
|
@cp-coder to Add Screenshot of Swagger Docs for the new Consultation API |
|
|
vigneshhari
requested changes
Sep 19, 2022
|
Let's not delete any fields immediately; let's keep a leeway of 1 month before deleting any data. remove any migration that deletes the columns. |
Codecov ReportBase: 56.08% // Head: 56.13% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #979 +/- ##
==========================================
+ Coverage 56.08% 56.13% +0.05%
==========================================
Files 175 178 +3
Lines 8617 8689 +72
Branches 1460 1478 +18
==========================================
+ Hits 4833 4878 +45
- Misses 3732 3759 +27
Partials 52 52
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
- Added facility & patient as ForeignKey field - Added has_allergy as BooleanField - Added family_details & allergies as TextField - Added blood_group as CharField
- Added has_allergy as BooleanField - Migrated blood_group & allergies fields from PatientRegistration Model
- Added import statement of PatientHealthDetails model - Removed blood_group field from PatientListSerializer - Removed allergies from exclude option of PatientListSerializer - Added serializer for patient health details
…lth details model
* Added function to migrate previous data to new Vaccine model
* Added serializer field for vaccination in health details serializer * Reformatted the code
* Added migrations for adding new notification events * Added new fields in notification generator for health details * Added notification generator in consultation serializers for health details
* Changed config file for flake8 to .flake8 * Removed all migrations for removing fields * Merged all the migrations for health details model into one migration * Refactored health details serializer * Reformatted the code
* Added deleted fields with `#deprecated` text * Reformatted the code
* Refactored the migrations * Updated migrations' dependencies * Added function to link health details with patient consultation
* Reverted back to medical history used * Removed new MedicalHistory and Disease model * Reformatted the code
* Switched to serializer to create/update health details record * Added create/update method in health details serializer * Added auth while creating health details record * Switched to validated_data for health details data * Reformatted the code
* Added vaccination in health details model as jsonfield * Added schema for vaccintion field * Removed vaccine model and its references
* Added vaccine, medical history nad disease model * Added the migrations with function to migrate the data * Added required serializers * Reformatted the code
|
SonarCloud Quality Gate failed.
|
|
Will open a new PR squashing the commits |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updates