update CUD operations to use _id

Author: tyouwei

peerprep-fe/src/app/(main)/components/problems/ProblemRow.tsx

@@ -21,7 +21,7 @@ export default function ProblemRow({ problem }: { problem: Problem }) {
   const handleProblemClick = async () => {
     // Fetch problem details here (replace with actual API call)
     const details: ProblemDialogData = {
-      question_id: problem.question_id,
+      _id: problem._id,
       title: problem.title,
       difficulty: problem.difficulty,
       description: problem.description,

peerprep-fe/src/app/(main)/components/problems/ProblemTable.tsx

@@ -29,8 +29,9 @@ export default function ProblemTable({ problems }: ProblemTableProps) {
           </tr>
         </thead>
         <tbody>
-          {filteredProblems.map((problem, index) => (
-            <ProblemRow key={index} problem={problem} />
+          {filteredProblems.map((problem) => (
+            // should not use index as key
+            <ProblemRow key={problem._id} problem={problem} />
           ))}
         </tbody>
       </table>

peerprep-fe/src/types/types.ts

@@ -1,5 +1,5 @@
 interface Problem {
-  question_id: number;
+  _id: number;
   title: string;
   difficulty: number;
   description: string;
@@ -11,7 +11,7 @@ interface Problem {
 }
 
 interface ProblemDialogData {
-  question_id: number;
+  _id: number;
   title: string;
   difficulty: number;
   description: string;

question-service/src/app.ts

@@ -4,9 +4,6 @@ import cors from 'cors';
 import * as dotenv from 'dotenv';
 import questionsRouter from './routes/questionsController';
 import userQuestionsRouter from './routes/userQuestionsController';
-import insertQuestionsRouter from './routes/insertQuestionsController';
-import updateQuestionsRouter from './routes/updateQuestionsController';
-import deleteQuestionsRouter from './routes/deleteQuestionsController';
 
 dotenv.config({ path: '.env.dev' });
 
@@ -30,9 +27,6 @@ app.use(
 // different routes
 app.use(`${apiVersion}/questions`, questionsRouter);
 app.use(`${apiVersion}/userquestions`, userQuestionsRouter);
-// app.use(`${apiVersion}/createquestion`, insertQuestionsRouter);
-// app.use(`${apiVersion}/updatequestion`, updateQuestionsRouter);
-// app.use(`${apiVersion}/deletequestion`, deleteQuestionsRouter);
 
 app.listen(PORT, () => {
   console.log(`Server is running on port ${PORT}`);

question-service/src/routes/questionsController.ts

@@ -1,6 +1,6 @@
 // src/routes/items.ts
 import express, { Request, Response } from 'express';
-import { Collection } from 'mongodb';
+import { Collection, ObjectId } from 'mongodb';
 import { connectToDB } from '../db/mongoClient';
 import { Questions } from '../models/types';
 
@@ -39,15 +39,18 @@ router.put('/:questionId', async (req: Request, res: Response) => {
     }
 
     const result = await questionsCollection.updateOne(
-      { _question_id: Number(questionId) },
-      { $set: updatedQuestion }
+      { _id: new ObjectId(questionId) },
+      { $set: updatedQuestion },
     );
 
     if (result.matchedCount === 0) {
       return res.status(404).json({ error: 'Question not found.' });
     }
 
-    res.status(200).json({ message: 'Question updated successfully' });
+    res.status(200).json({
+      message: 'Question updated successfully',
+      data: [{ _id: questionId }],
+    });
   } catch (error) {
     console.error('Error updating question:', error);
     res.status(500).json({ error: 'Failed to update question' });
@@ -58,11 +61,19 @@ router.put('/:questionId', async (req: Request, res: Response) => {
 router.post('/', async (req: Request, res: Response) => {
   try {
     const newQuestion: Questions = req.body; // Assume the body contains a question object
-    
-    const { _question_id, difficulty, description, examples, constraints, tags, title_slug, title, pictures } = newQuestion;
+
+    const {
+      difficulty,
+      description,
+      examples,
+      constraints,
+      tags,
+      title_slug,
+      title,
+      pictures,
+    } = newQuestion;
 
     if (
-      typeof _question_id !== 'number' ||
       typeof difficulty !== 'number' ||
       typeof description !== 'string' ||
       !Array.isArray(examples) ||
@@ -71,12 +82,17 @@ router.post('/', async (req: Request, res: Response) => {
       typeof title_slug !== 'string' ||
       typeof title !== 'string'
     ) {
-      return res.status(400).json({ error: 'Invalid question data. Please check your input.' });
+      return res
+        .status(400)
+        .json({ error: 'Invalid question data. Please check your input.' });
     }
 
     const result = await questionsCollection.insertOne(newQuestion);
 
-    res.status(201).json({ message: 'Question inserted successfully', questionId: result.insertedId });
+    res.status(201).json({
+      message: 'Question inserted successfully',
+      data: [{ _id: result.insertedId }],
+    });
   } catch (error) {
     console.error('Error inserting question:', error);
     res.status(500).json({ error: 'Failed to insert question' });
@@ -88,13 +104,22 @@ router.delete('/:questionId', async (req: Request, res: Response) => {
   const { questionId } = req.params;
 
   try {
-    const result = await questionsCollection.deleteOne({ _question_id: Number(questionId) });
+    const result = await questionsCollection.deleteOne({
+      _id: new ObjectId(questionId),
+    });
 
     if (result.deletedCount === 0) {
       return res.status(404).json({ error: 'Question not found.' });
     }
 
-    res.status(200).json({ message: 'Question deleted successfully' });
+    res.status(200).json({
+      message: 'Question deleted successfully',
+      data: [
+        {
+          _id: questionId,
+        },
+      ],
+    });
   } catch (error) {
     console.error('Error deleting question:', error);
     res.status(500).json({ error: 'Failed to delete question' });