Fix 'excused' attendance status saving error

- Added 'excused' to AttendanceRecord interface type definition
- Updated attendance points logic to award points for 'excused' status
- Created migration to add 'excused' to attendance table CHECK constraint
- Updated database trigger to handle 'excused' status for attendance points
- Excused attendance now counts as present for point tracking purposes

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: Zeff01

apps/codebility/app/home/my-team/[projectId]/actions.ts

@@ -48,7 +48,7 @@ interface AttendanceRecord {
   codev_id: string;
   project_id: string;
   date: string;
-  status: "present" | "absent" | "late" | "holiday" | "weekend";
+  status: "present" | "absent" | "late" | "holiday" | "weekend" | "excused";
   check_in?: string;
   check_out?: string;
   notes?: string;
@@ -97,11 +97,11 @@ export async function saveAttendance(record: AttendanceRecord) {
     }
 
     // Manually update attendance points (in case trigger isn't working)
-    if (record.status === "present" || record.status === "late") {
+    if (record.status === "present" || record.status === "late" || record.status === "excused") {
       await updateAttendancePoints(record.codev_id, 2);
-    } else if (existing && (existing.status === "present" || existing.status === "late") && 
-               record.status !== "present" && record.status !== "late") {
-      // Deduct points if changing from present/late to absent
+    } else if (existing && (existing.status === "present" || existing.status === "late" || existing.status === "excused") && 
+               record.status !== "present" && record.status !== "late" && record.status !== "excused") {
+      // Deduct points if changing from present/late/excused to absent
       await updateAttendancePoints(record.codev_id, -2);
     }
 

apps/codebility/supabase/migrations/20250119_add_excused_status_to_attendance.sql

@@ -0,0 +1,46 @@
+-- Add 'excused' as a valid status to the attendance table
+ALTER TABLE attendance DROP CONSTRAINT IF EXISTS attendance_status_check;
+ALTER TABLE attendance ADD CONSTRAINT attendance_status_check 
+  CHECK (status IN ('present', 'absent', 'late', 'holiday', 'weekend', 'excused'));
+
+-- Update the trigger to handle excused status for attendance points
+-- Excused attendance should count as present for attendance tracking purposes
+CREATE OR REPLACE FUNCTION update_attendance_points_on_attendance_change()
+RETURNS TRIGGER AS $$
+BEGIN
+  -- Handle INSERT or UPDATE to present/late/excused
+  IF (TG_OP = 'INSERT' AND NEW.status IN ('present', 'late', 'excused')) OR
+     (TG_OP = 'UPDATE' AND OLD.status NOT IN ('present', 'late', 'excused') AND NEW.status IN ('present', 'late', 'excused')) THEN
+    -- Add 2 points
+    INSERT INTO attendance_points (codev_id, points, last_updated)
+    VALUES (NEW.codev_id, 2, CURRENT_DATE)
+    ON CONFLICT (codev_id) 
+    DO UPDATE SET 
+      points = attendance_points.points + 2,
+      last_updated = CURRENT_DATE,
+      updated_at = NOW();
+      
+  -- Handle UPDATE from present/late/excused to other status
+  ELSIF TG_OP = 'UPDATE' AND OLD.status IN ('present', 'late', 'excused') AND NEW.status NOT IN ('present', 'late', 'excused') THEN
+    -- Deduct 2 points
+    UPDATE attendance_points 
+    SET 
+      points = GREATEST(0, points - 2),
+      last_updated = CURRENT_DATE,
+      updated_at = NOW()
+    WHERE codev_id = NEW.codev_id;
+    
+  -- Handle DELETE of present/late/excused record
+  ELSIF TG_OP = 'DELETE' AND OLD.status IN ('present', 'late', 'excused') THEN
+    -- Deduct 2 points
+    UPDATE attendance_points 
+    SET 
+      points = GREATEST(0, points - 2),
+      last_updated = CURRENT_DATE,
+      updated_at = NOW()
+    WHERE codev_id = OLD.codev_id;
+  END IF;
+  
+  RETURN NEW;
+END;
+$$ LANGUAGE plpgsql;