Merge pull request #249 from cmu15122/alexb/filtering

Filtering fixed
cmu15122 · Oct 9, 2023 · 6e9785e · 6e9785e
2 parents 8620d66 + 317a48e
commit 6e9785e
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/client/src/components/home/ta/StudentEntries.tsx b/client/src/components/home/ta/StudentEntries.tsx
@@ -95,17 +95,17 @@ export default function StudentEntries(props) {
 
   useEffect(() => {
     setIsHelping(false);
-    for (const [index, student] of allStudents.entries()) {
+    for (const [index, student] of filteredStudents.entries()) {
       if (student.status === StudentStatusValues.BEING_HELPED && student.helpingTAInfo?.taAndrewID === userData.andrewID) {
         setHelpIdx(index);
         setIsHelping(true);
       }
     }
-  }, [allStudents, userData.andrewID]);
+  }, [filteredStudents, userData.andrewID]);
 
   const handleClickHelp = (index) => {
     HomeService.helpStudent(JSON.stringify({
-      andrewID: allStudents[index].andrewID,
+      andrewID: filteredStudents[index].andrewID,
     })).then((res) => {
       if (res.status === 200) {
         setHelpIdx(index);
@@ -116,7 +116,7 @@ export default function StudentEntries(props) {
 
   const handleCancel = (index) => {
     HomeService.unhelpStudent(JSON.stringify({
-      andrewID: allStudents[index].andrewID,
+      andrewID: filteredStudents[index].andrewID,
     })).then((res) => {
       if (res.status === 200) {
         setHelpIdx(-1);
@@ -127,15 +127,15 @@ export default function StudentEntries(props) {
 
   const removeStudent = (index, doneHelping) => {
     HomeService.removeStudent(JSON.stringify({
-      andrewID: allStudents[index].andrewID,
+      andrewID: filteredStudents[index].andrewID,
       doneHelping: doneHelping,
     }));
   };
 
   const handleClickUnfreeze = (index) => {
     setIsHelping(false);
     setHelpIdx(-1);
-    allStudents[index].status = StudentStatusValues.WAITING;
+    filteredStudents[index].status = StudentStatusValues.WAITING;
   };
 
   /* END QUEUE LOGIC */