fix: adds doctor and staff fields in location model

[aeswibon]

Feature Request

Proposed Changes

• Adds duty_staff field to asset location model and serializer
• Adds home_facility type filter in UserFilter

Associated PR

• Required by fix: Duty doctors and Staff for each Location in a Facility care_fe#5370

Merge Checklist

• Tests added/fixed
• Update docs in /docs
• Linting Complete

@coronasafe/code-reviewers

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (0daef7f) 60.02% compared to head (6a693e0) 60.05%.

❗ Current head 6a693e0 differs from pull request most recent head 8b9d4d1. Consider uploading reports for the commit 8b9d4d1 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1264      +/-   ##
==========================================
+ Coverage   60.02%   60.05%   +0.02%     
==========================================
  Files         208      208              
  Lines       11319    11322       +3     
  Branches     1606     1606              
==========================================
+ Hits         6794     6799       +5     
+ Misses       4301     4300       -1     
+ Partials      224      223       -1     

Files	Coverage Δ
care/facility/api/serializers/asset.py	84.39% <100.00%> (+0.09%)	⬆️
care/facility/api/viewsets/facility_users.py	93.75% <100.00%> (+0.20%)	⬆️
care/facility/models/asset.py	95.28% <100.00%> (+0.04%)	⬆️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nihal467]

LGTM

[sainak]

Validation required for limiting the number of staff assigned

[vigneshhari]

I don't really see a use for this. how are users going to be added in the M2M model?

[nihal467]

@gigincg can you comment on this ?

[gigincg]

@cp-coder @sainak Rephrasing @vigneshhari 's comment. I don't see CRUD APIs for duty_doctors

I have added the limit

[aeswibon]

Discussed in the EOD. Closing the PR as I will be creating a new endpoint that will do the same functionality