Merge pull request #177 from PeerPrep/kevin/new

feat: add error msg for duplicate qn names

Author: sivayogasubramanian

frontend/src/app/admin/question/page.tsx

@@ -198,7 +198,6 @@ const QuestionPage = () => {
   });
 
   const allQuestionsFiltered = useMemo(() => {
-    console.log(allQuestions?.payload);
     if (Array.isArray(allQuestions?.payload)) {
       return allQuestions?.payload.filter((question) => {
         return (
@@ -212,8 +211,6 @@ const QuestionPage = () => {
     }
   }, [allQuestions?.payload, searchValue, filterQnType]);
 
-  console.log({ allQuestionsFiltered });
-
   const deleteQuestionMutation = useMutation(
     async (questionId: string) => deleteQuestionUrl(questionId),
     {

frontend/src/app/api/index.ts

@@ -60,7 +60,6 @@ export const fetchAllQuestionsDoneByUser = async () => {
   );
 
   const questionIds = payload.map((ele: any) => ele.questionId).join("-");
-  console.log({ questionIds });
   // console.log({ res });
   const questions = await FetchAuth.fetch(
     `${API_URL}/questions/group/${questionIds}`,
@@ -120,7 +119,14 @@ export const createQuestionUrl = async (newQuestion: QuestionType) => {
       "Content-Type": "application/json",
     },
     body: JSON.stringify(newQuestion),
-  }).then((res) => res.json());
+  })
+    .then((res) => res.json())
+    .then((res) => {
+      if (res.statusMessage?.type?.toLowerCase() === "error") {
+        throw Error();
+      }
+      return res;
+    });
 };
 
 export const updateQuestionUrl = async (updatedQuestion: QuestionType) => {
@@ -130,7 +136,14 @@ export const updateQuestionUrl = async (updatedQuestion: QuestionType) => {
       "Content-Type": "application/json",
     },
     body: JSON.stringify(updatedQuestion),
-  }).then((res) => res.json());
+  })
+    .then((res) => res.json())
+    .then((res) => {
+      if (res.statusMessage?.type?.toLowerCase() === "error") {
+        throw Error();
+      }
+      return res;
+    });
 };
 
 export const deleteQuestionUrl = async (questionId: string) => {

frontend/src/app/components/matching/MatchingPage.tsx

@@ -139,8 +139,6 @@ const MatchingPage = () => {
     return fetchAllQuestionsDoneByUser();
   });
 
-  console.log({ allQuestions });
-
   return (
     <>
       <main className="flex h-full flex-col items-center justify-center">

frontend/src/app/components/modal/AddQuestionModal.tsx

@@ -70,7 +70,7 @@ const AddQuestionModal = ({
       onError: (e) => {
         api.open({
           type: "error",
-          content: "Failed to add question!",
+          content: "Failed to add question due to same name!",
         });
       },
     },
@@ -92,8 +92,7 @@ const AddQuestionModal = ({
       description: e.currentTarget.description.value,
     };
 
-    console.log("Form Submission Data:", submissionData);
-    createQuestionMutation.mutate(submissionData);
+    createQuestionMutation && createQuestionMutation.mutate(submissionData);
     e.currentTarget.reset();
     setDefaultValues();
     closeModal("my_modal_1");

frontend/src/app/components/modal/EditQuestionModal.tsx

@@ -69,7 +69,6 @@ const EditQuestionModal = ({
         successCallback();
       },
       onError: (e) => {
-        console.log({ e });
         api.open({
           type: "error",
           content: "Failed to add question due to having same question name!",

frontend/src/app/matching/page.tsx

@@ -58,7 +58,6 @@ const roomPage = () => {
     );
   }
 
-  console.log(isMatched);
   if (isMatched === "UNMATCHED") {
     return <MatchingPage />;
   }