Merge pull request #14 from Bunk-Mate/13-time-table-update-failure

fix: replace null with empty string before timetable patch and add gaurds for timetable load

Author: amansxcalibur

app/dashboard/(timetable)/edit/page.jsx

@@ -57,6 +57,7 @@ export default function EditTable() {
    }, [popupDecision]);
 
    const handleSaveTimetable = () => {
+      const nullLessTableData = removeNull(tableData)
       addNotification('Request sent. Please wait.');
       const header = {
          Authorization:
@@ -65,7 +66,7 @@ export default function EditTable() {
       axios
          .patch(
             API_BASE_URL + '/collection',
-            { courses_data: tableData },
+            { courses_data: nullLessTableData },
             { headers: header }
          )
          .then((response) => {
@@ -316,3 +317,15 @@ function getOptions({ timetable }) {
    }
    return options;
 }
+
+function removeNull(tableData) {
+   var nullLessTableData = tableData;
+   for (let i = 0; i < tableData.length; i++) {
+      for (let j = 0; j < 5; j++) {
+         if (nullLessTableData[i][j] == null) {
+            nullLessTableData[i][j] = '';
+         }
+      }
+   }
+   return nullLessTableData;
+}

components/shared_timetable/load_timetable.jsx

@@ -10,6 +10,10 @@ export default async function LoadTimetable({
       Authorization:
          'Token ' + JSON.parse(localStorage.getItem(ACCESS_TOKEN_NAME)),
    };
+   if (selectedOption == undefined) {
+      addNotification('Choose an option');
+      return;
+   }
    const payload = { copy_id: selectedOption.id };
    try {
       addNotification('Loading Timetable...');