Skip to content

Commit

Permalink
fix(holidays): add holiday admins to holidays request notifications
Browse files Browse the repository at this point in the history
  • Loading branch information
pablolmedorado committed Jun 19, 2021
1 parent a00f502 commit f9ee27a
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion backend/work_organization/api/viewsets.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
from datetime import date

from django.contrib.auth import get_user_model
from django.contrib.auth.models import Permission
from django.db import transaction
from django.db.models import Count, F, Min
from django.utils import timezone
Expand Down Expand Up @@ -147,9 +148,15 @@ def request(self, request, *args, **kwargs):
if holiday:
base_queryset.filter(pk=holiday.pk).update(planned_date=requested_date)
holiday_pks.append(holiday.pk)

global_admin_users = get_user_model().objects.admins().exclude(pk=request.user.pk)
change_holidays_permission = Permission.objects.get_by_natural_key(
"change_holiday", "work_organization", "holiday"
)
holiday_admin_users = change_holidays_permission.user_set.exclude(pk=request.user.pk)
notify.send(
sender=request.user,
recipient=get_user_model().objects.admins().exclude(pk=request.user.pk),
recipient=global_admin_users.union(holiday_admin_users),
verb=f"ha solicitado {len(requested_dates)} días de vacaciones",
)

Expand Down

0 comments on commit f9ee27a

Please sign in to comment.