🎨 chore: Make the attendanceRecords destructured from the request bod…

…y & add some comments
CMP26Projects · Dec 29, 2023 · 3dfe2cf · 3dfe2cf
1 parent 4f9bcd8
commit 3dfe2cf
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/server/controllers/captainAttendance.controller.js b/server/controllers/captainAttendance.controller.js
@@ -3,7 +3,8 @@ import db from "../database/db.js"
 const captainAttendanceController = {
     upsertAttendance: async (req, res) => {
         try {
-            const  attendanceRecords  = req.body.body
+            // NOTE: attendanceRecords must be sent as an array of objects in the request body
+            const  { attendanceRecords }  = req.body
 
             if (attendanceRecords === 0) {
                 return res.status(404).json({
@@ -14,6 +15,7 @@ const captainAttendanceController = {
             const weekNumber = attendanceRecords[0].weekNumber;
             const termNumber = attendanceRecords[0].termNumber;
 
+            // Query to check if there are an exisiting records for this week & term or not
             const prevRecords = await db.query(`
                 SELECT *
                 FROM "CaptainAttendance"

diff --git a/server/controllers/scoutAttendance.controller.js b/server/controllers/scoutAttendance.controller.js
@@ -6,7 +6,8 @@ const scoutAttendanceController = {
     // @access  Private
     upsertAttendance: async (req, res) => {
         try {
-            const  attendanceRecords  = req.body.body
+            // NOTE: attendanceRecords must be sent as an array of objects in the request body
+            const { attendanceRecords }  = req.body
 
             if (attendanceRecords === 0) {
                 return res.status(404).json({
@@ -16,7 +17,8 @@ const scoutAttendanceController = {
 
             const weekNumber = attendanceRecords[0].weekNumber;
             const termNumber = attendanceRecords[0].termNumber;
-
+
+            // Query to check if there are an exisiting records for this week & term or not
             const prevRecords = await db.query(`
                 SELECT *
                 FROM "ScoutAttendance"