Skip to content
Snippets Groups Projects
Commit 6ad0b9f9 authored by Lorenzo Conti's avatar Lorenzo Conti
Browse files

Merge branch 'fix/person-no-default-avail' into 'main'

If no owner availability is given, assume avail for full event

See merge request !14
parents fe24cd4e dafdcf59
No related branches found
No related tags found
1 merge request!14If no owner availability is given, assume avail for full event
Pipeline #271613 passed
......@@ -1006,7 +1006,8 @@ class AKSlot(models.Model):
ak_time_constraints = []
def _owner_time_constraints(owner: AKOwner):
if Availability.is_event_covered(self.event, owner.availabilities.all()):
owner_avails = owner.availabilities.all()
if not owner_avails or Availability.is_event_covered(self.event, owner_avails):
return []
return [f"availability-person-{owner.pk}"]
......
......@@ -479,6 +479,10 @@ class JSONExportTest(TestCase):
for owner in slot.ak.owners.all():
# restricted owner availability
if not owner.availabilities.all():
# no availability for owner -> assume full event is covered
continue
if not Availability.is_event_covered(
slot.event, owner.availabilities.all()
):
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment