Bug fix questionId

backend/question-service/src/models/question.ts

@@ -1,7 +1,7 @@
 import { Schema, model, Document } from "mongoose";
 
 interface IQuestion extends Document {
-  questionId: string;
+  questionId: Number;
   title: string;
   description: string;
   categories: string[];
@@ -10,7 +10,7 @@ interface IQuestion extends Document {
 }
 
 const questionSchema = new Schema<IQuestion>({
-  questionId: { type: String, required: true, unique: true },
+  questionId: { type: Number, required: true, unique: true },
   title: { type: String, required: true },
   description: { type: String, required: true },
   categories: { type: [String], required: true },

backend/question-service/src/routes/questionRoutes.ts

@@ -5,7 +5,8 @@ const router: Router = Router();
 
 // Create a new question (POST)
 router.post("/", async (req: Request, res: Response) => {
-  const { questionId, title, description, categories, complexity, link } = req.body;
+  const { questionId, title, description, categories, complexity, link } =
+    req.body;
 
   try {
     const newQuestion = new Question({
@@ -36,7 +37,10 @@ router.get("/", async (req: Request, res: Response) => {
 // Get a single question by ID (GET)
 router.get("/:id", async (req: Request, res: Response) => {
   try {
-    const question = await Question.findOne({ questionId: req.params.id });
+    const id = Number(req.params.id);
+    const question = await Question.findOne({
+      questionId: id,
+    });
     if (!question) return res.status(404).json({ error: "Question not found" });
     res.json(question);
   } catch (err) {
@@ -49,12 +53,14 @@ router.put("/:id", async (req: Request, res: Response) => {
   const { title, description, categories, complexity } = req.body;
 
   try {
+    const id = Number(req.params.id);
     const updatedQuestion = await Question.findOneAndUpdate(
-      { questionId: req.params.id },
+      { questionId: id },
       { title, description, categories, complexity },
       { new: true }
     );
-    if (!updatedQuestion) return res.status(404).json({ error: "Question not found" });
+    if (!updatedQuestion)
+      return res.status(404).json({ error: "Question not found" });
     res.json(updatedQuestion);
   } catch (err) {
     res.status(500).json({ error: (err as Error).message });
@@ -64,10 +70,12 @@ router.put("/:id", async (req: Request, res: Response) => {
 // Delete a question by ID (DELETE)
 router.delete("/:id", async (req: Request, res: Response) => {
   try {
+    const id = Number(req.params.id);
     const deletedQuestion = await Question.findOneAndDelete({
-      questionId: req.params.id,
+      questionId: id,
     });
-    if (!deletedQuestion) return res.status(404).json({ error: "Question not found" });
+    if (!deletedQuestion)
+      return res.status(404).json({ error: "Question not found" });
     res.json({ message: "Question deleted successfully" });
   } catch (err) {
     res.status(500).json({ error: (err as Error).message });

frontend/src/components/QuestionDialog/QuestionDialog.tsx

@@ -17,14 +17,14 @@ import {
 const QuestionDialog = (props: {
   isAddNew: boolean;
   isOpen: boolean;
-  id: string;
+  id: number;
   title: string;
   description: string;
   categories: string[];
   complexity: QuestionComplexity;
   link: string;
   setIsOpen: (isOpen: boolean) => void;
-  setId: (id: string) => void;
+  setId: (id: number) => void;
   setMainDialogTitle: (title: string) => void;
   setDescription: (description: string) => void;
   setCategories: (categories: string[]) => void;
@@ -103,7 +103,7 @@ const QuestionDialog = (props: {
             label="ID"
             type="text"
             value={id}
-            onChange={(e) => setId(e.target.value)}
+            onChange={(e) => setId(Number(e.target.value))}
           />
           <TextField
             className="QuestionDialog-input QuestionDialog-input-title"

frontend/src/components/QuestionList/QuestionList.tsx

@@ -26,7 +26,7 @@ const QuestionList = (): ReactElement => {
   // Shared states for adding or editing question
   const [isAddNew, setIsAddNew] = useState<boolean>(true);
   const [isQuestionDialogOpen, setIsQuestionDialogOpen] = useState<boolean>(false);
-  const [newQuestionId, setNewQuestionId] = useState<string>("");
+  const [newQuestionId, setNewQuestionId] = useState<number>(0);
   const [newQuestionTitle, setNewQuestionTitle] = useState<string>("");
   const [newQuestionDescription, setNewQuestionDescription] = useState<string>("");
   const [newQuestionCategories, setNewQuestionCategories] = useState<string[]>([]);
@@ -35,7 +35,7 @@ const QuestionList = (): ReactElement => {
 
   const openQuestionDialog = (question: Question | null) => {
     setIsAddNew(!question);
-    setNewQuestionId(question?.questionId ?? "");
+    setNewQuestionId(question?.questionId ?? 0);
     setNewQuestionTitle(question?.title ?? "");
     setNewQuestionDescription(question?.description ?? "");
     setNewQuestionCategories(question?.categories ?? []);
@@ -103,8 +103,8 @@ const QuestionList = (): ReactElement => {
 
     const sortedQuestions = [...questions].sort((a, b) => {
       if (key === "questionId") {
-        const aId = parseInt(a[key]);
-        const bId = parseInt(b[key]);
+        const aId = a[key];
+        const bId = b[key];
         return order === "asc" ? aId - bId : bId - aId;
       }
 

frontend/src/constants/mock_questions.ts

@@ -2,7 +2,7 @@ import { Question } from "../models/question.model";
 
 export const mockQuestions: Question[] = [
   {
-    questionId: "1",
+    questionId: 1,
     title: "What is a Binary Search Tree?",
     description:
       "A binary search tree is a data structure that facilitates fast lookup, addition, and removal of items.",
@@ -11,15 +11,15 @@ export const mockQuestions: Question[] = [
     link: "",
   },
   {
-    questionId: "2",
+    questionId: 2,
     title: "Explain the concept of closures in JavaScript",
     description: "Closures are a way to access variables defined outsquestionIde of a function's scope.",
     categories: ["JavaScript", "Functional Programming"],
     complexity: "Hard",
     link: "",
   },
   {
-    questionId: "3",
+    questionId: 3,
     title: "What is polymorphism in OOP?",
     description:
       "Polymorphism is the ability of different objects to respond in different ways to the same method call.",
@@ -28,7 +28,7 @@ export const mockQuestions: Question[] = [
     link: "",
   },
   {
-    questionId: "4",
+    questionId: 4,
     title: "Reverse a String",
     description: "Write a function that reverses a string. The input string is given as an array of characters.",
     categories: ["Strings", "Algorithms"],

frontend/src/models/question.model.ts

@@ -1,7 +1,7 @@
 export type QuestionComplexity = "Easy" | "Medium" | "Hard";
 
 export interface Question {
-  questionId: string;
+  questionId: number;
   title: string;
   description: string;
   categories: string[];

frontend/src/services/question.service.ts

@@ -9,7 +9,7 @@ export default class QuestionService {
   }
 
   static async addQuestion(
-    id: string,
+    id: number,
     title: string,
     description: string,
     categoriesString: string,
@@ -22,7 +22,7 @@ export default class QuestionService {
   }
 
   static async editQuestion(
-    id: string,
+    id: number,
     title: string,
     description: string,
     categoriesString: string,
@@ -34,7 +34,7 @@ export default class QuestionService {
     return response.data;
   }
 
-  static async deleteQuestion(id: string): Promise<any> {
+  static async deleteQuestion(id: number): Promise<any> {
     const response = await axios.delete(`/api/questions/${id}`);
     return response.data;
   }

frontend/src/util/question.helper.ts

@@ -44,9 +44,8 @@ const verifyLength = (value: string, field: QuestionValidationFields): void => {
   }
 };
 
-const verifyId = (id: string): string => {
-  verifyLength(id, "id");
-  return id.trim();
+const verifyId = (id: number): number => {
+  return id;
 };
 
 const verifyTitle = (title: string): string => {
@@ -86,7 +85,7 @@ const verifyLink = (link: string): string => {
 };
 
 export const verifyNewQuestion = (
-  id: string,
+  id: number,
   title: string,
   description: string,
   categoriesString: string,