Fixed: Added occupation field for text support

Open aeswibon opened this issue 1 year ago • 2 comments

Updates

[x] Fixes #806
[x] Removed Occupation Choices from PatientMetaInfo model
[x] Altered the occupation field from IntegerField to CharField
[x] Removed occupation field as ChoiceField from PatientMetaInfoSerializer
[x] Added migration to alter the schema for occupation field
[x] Added migration to retain all the data

Oct 09 '22 04:10 aeswibon

Codecov Report

Base: 56.06% // Head: 56.02% // Decreases project coverage by -0.04% :warning:

Coverage data is based on head (c4a376a) compared to base (0beaa72). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1058      +/-   ##
==========================================
- Coverage   56.06%   56.02%   -0.05%     
==========================================
  Files         175      175              
  Lines        8629     8620       -9     
  Branches     1462     1475      +13     
==========================================
- Hits         4838     4829       -9     
  Misses       3739     3739              
  Partials       52       52

Impacted Files	Coverage Δ
care/facility/api/serializers/patient.py	`55.81% <100.00%> (-0.21%)`	:arrow_down:
care/facility/models/patient.py	`92.59% <100.00%> (-0.27%)`	:arrow_down:
care/users/api/viewsets/change_password.py	`61.53% <0.00%> (+1.53%)`	:arrow_up: