Skip to content
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.

Sign up for GitHub

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

chore: Add total doctors count to hospital doctor list response #2629

Closed
wants to merge 7 commits into from
Closed

chore: Add total doctors count to hospital doctor list response #2629

Changes from 2 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
42350b4
chore: Add total doctors count to hospital doctor list response
shauryag2002 Dec 2, 2024
5f85647
Handle potential None value from aggregation
shauryag2002 Dec 2, 2024
7074cf0
Merge branch 'ohcnetwork:develop' into backend/auto-scroll-card-number
shauryag2002 Dec 7, 2024
83a2f2b
Reverting hospital_doctor.py
shauryag2002 Dec 7, 2024
bfd4f75
Update hospital_doctor.py
shauryag2002 Dec 7, 2024
fa6f6bd
Update hospital_doctor.py
shauryag2002 Dec 7, 2024
80aefde
Update hospital_doctor.py
shauryag2002 Dec 7, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions care/facility/api/viewsets/hospital_doctor.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
from rest_framework.generics import get_object_or_404
from rest_framework.mixins import ListModelMixin
from rest_framework.permissions import IsAuthenticated
from django.db.models import Sum

from care.facility.api.serializers.hospital_doctor import HospitalDoctorSerializer
from care.facility.api.viewsets import FacilityBaseViewset
Expand Down Expand Up @@ -42,5 +43,11 @@ def get_facility(self):
facility_qs.filter(users__id__exact=self.request.user.id)
return get_object_or_404(facility_qs)

def list(self, request, *args, **kwargs):
response = super().list(request, *args, **kwargs)
total_doctors = self.get_queryset().aggregate(total_doctors=Sum('count'))['total_doctors']
response.data["total_doctors"] = total_doctors or 0
return response

def perform_create(self, serializer):
serializer.save(facility=self.get_facility())