wip

uktrade · Nov 19, 2024 · 6c7e5e7 · 6c7e5e7
1 parent 3fd969a
commit 6c7e5e7
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 27 deletions.
diff --git a/gifthospitality/migrations/0007_data_test_multiple_concurrent.py b/gifthospitality/migrations/0007_data_test_multiple_concurrent.py
diff --git a/payroll/models.py b/payroll/models.py
@@ -43,7 +43,7 @@ class Employee(models.Model):
     fte = models.FloatField(default=1.0)
     assignment_status = models.CharField(max_length=32)
 
-    # TODO: Missing fields from Admin Tool
+    # TODO: Missing fields from Admin Tool which aren't required yet.
     # EU/Non-EU (from programme code model)
 
     objects = EmployeeQuerySet.as_manager()
@@ -94,7 +94,8 @@ class Meta:
     period_10 = models.BooleanField(default=True)
     period_11 = models.BooleanField(default=True)
     period_12 = models.BooleanField(default=True)
-    # FIXME
+
+    # TODO: Missing fields from Admin Tool which aren't required yet.
     # capital (Real colour of money)
     # recharge = models.CharField(max_length=50, null=True, blank=True)
     # recharge_reason = models.CharField(max_length=100, null=True, blank=True)

diff --git a/payroll/services/payroll.py b/payroll/services/payroll.py
@@ -99,6 +99,7 @@ def get_payroll_data(
             budget_type=obj.programme_code.budget_type.budget_type_display,
             assignment_status=obj.assignment_status,
             basic_pay=obj.basic_pay,
+            # `first` is OK as there should only be one `pay_periods` with the filters.
             pay_periods=obj.pay_periods.first().periods,
         )