Add update question API in QuestionService

Link frontend form for updating question to QuestionService

Author: szelongq

QuestionService/javascript/index.js

@@ -15,6 +15,7 @@ app.get("/", (req, res) => {
 });
 app.get("/questions", question_controller_1.handleGetQuestions);
 app.post("/questions", question_controller_1.handleAddQuestion);
+app.put("/questions/:questionId", question_controller_1.handleUpdateQuestion);
 app.delete("/questions/:questionId", question_controller_1.handleDeleteQuestion);
 app.listen(port, () => {
     console.log(`Question Service listening on port ${port}`);

QuestionService/javascript/question/question.controller.js

@@ -9,7 +9,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.handleAddQuestion = exports.handleDeleteQuestion = exports.handleGetQuestions = void 0;
+exports.handleAddQuestion = exports.handleUpdateQuestion = exports.handleDeleteQuestion = exports.handleGetQuestions = void 0;
 const question_service_1 = require("./question.service");
 const firestore_1 = require("firebase/firestore");
 function handleGetQuestions(req, res) {
@@ -74,6 +74,30 @@ function handleDeleteQuestion(req, res) {
     });
 }
 exports.handleDeleteQuestion = handleDeleteQuestion;
+function handleUpdateQuestion(req, res) {
+    return __awaiter(this, void 0, void 0, function* () {
+        const questionId = req.params.questionId;
+        try {
+            const { title, tags, categories, constraints, difficulty, description, examples, } = req.body;
+            console.log(`updating question ${questionId}: ${title}`);
+            const question = yield (0, question_service_1.updateQuestion)(questionId, {
+                title: title,
+                tags: tags,
+                categories: categories,
+                constraints: constraints,
+                difficulty: difficulty,
+                description: description,
+                examples: examples,
+            });
+            res.status(200).send(question);
+        }
+        catch (err) {
+            console.log(err);
+            res.status(500).send(err);
+        }
+    });
+}
+exports.handleUpdateQuestion = handleUpdateQuestion;
 function handleAddQuestion(req, res) {
     return __awaiter(this, void 0, void 0, function* () {
         try {

QuestionService/javascript/question/question.service.js

@@ -9,7 +9,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.addQuestion = exports.deleteQuestion = exports.db = void 0;
+exports.addQuestion = exports.updateQuestion = exports.deleteQuestion = exports.db = void 0;
 const app_1 = require("firebase/app");
 const firestore_1 = require("firebase/firestore");
 const firebase_config_1 = require("../firebase/firebase.config");
@@ -28,6 +28,23 @@ function deleteQuestion(questionId) {
     });
 }
 exports.deleteQuestion = deleteQuestion;
+function updateQuestion(questionId, question) {
+    return __awaiter(this, void 0, void 0, function* () {
+        try {
+            let questionDoc = question;
+            const docRef = (0, firestore_1.doc)(exports.db, "questions", questionId);
+            yield (0, firestore_1.setDoc)(docRef, questionDoc);
+            for (let i = 0; i < question.examples.length; i++) {
+                const add = (0, firestore_1.setDoc)((0, firestore_1.doc)(docRef, "examples", (i + 1).toString()), question.examples[i]);
+            }
+            return Promise.resolve(question);
+        }
+        catch (error) {
+            return Promise.reject(error);
+        }
+    });
+}
+exports.updateQuestion = updateQuestion;
 function addQuestion(question) {
     return __awaiter(this, void 0, void 0, function* () {
         try {

QuestionService/src/index.ts

@@ -1,7 +1,7 @@
 import express from "express";
 import cors from "cors";
 
-import { handleGetQuestions, handleAddQuestion, handleDeleteQuestion } from "./question/question.controller";
+import { handleGetQuestions, handleAddQuestion, handleUpdateQuestion, handleDeleteQuestion } from "./question/question.controller";
 const app = express();
 const port = 3002;  
 
@@ -15,6 +15,7 @@ app.get("/", (req, res) => {
 
 app.get("/questions", handleGetQuestions);
 app.post("/questions", handleAddQuestion)
+app.put("/questions/:questionId", handleUpdateQuestion)
 app.delete("/questions/:questionId", handleDeleteQuestion)
 
 app.listen(port, () => {

QuestionService/src/question/question.controller.ts

@@ -1,5 +1,5 @@
 import { Request, Response } from "express";
-import {addQuestion, db, deleteQuestion} from "./question.service";
+import {addQuestion, db, deleteQuestion, updateQuestion} from "./question.service";
 import {
   getDocs,
   collection,
@@ -87,6 +87,35 @@ export async function handleDeleteQuestion(req: Request, res: Response) {
   }
 }
 
+export async function handleUpdateQuestion(req: Request, res: Response) {
+  const questionId = req.params.questionId;
+  try {
+    const {
+      title,
+      tags,
+      categories,
+      constraints,
+      difficulty,
+      description,
+      examples,
+    } = req.body;
+    console.log(`updating question ${questionId}: ${title}`);
+    const question = await updateQuestion(questionId,{
+      title: title,
+      tags: tags,
+      categories: categories,
+      constraints: constraints,
+      difficulty: difficulty,
+      description: description,
+      examples: examples,
+    });
+    res.status(200).send(question);
+  } catch (err) {
+    console.log(err);
+    res.status(500).send(err);
+  }
+}
+
 export async function handleAddQuestion(req: Request, res: Response) {
   try {
     const {

QuestionService/src/question/question.service.ts

@@ -1,5 +1,5 @@
 import { initializeApp } from "firebase/app";
-import {deleteDoc, doc, getFirestore, setDoc} from "firebase/firestore";
+import {addDoc, collection, deleteDoc, doc, getFirestore, setDoc} from "firebase/firestore";
 import { firebaseConfig } from "../firebase/firebase.config"
 
 initializeApp(firebaseConfig);
@@ -30,10 +30,10 @@ export async function deleteQuestion(questionId: string) {
   }
 }
 
-export async function addQuestion(question: Question): Promise<Question> {
+export async function updateQuestion(questionId: string, question: Question): Promise<Question> {
   try {
     let questionDoc: Omit<Question, "examples"> = question
-    const docRef = doc(db, "questions", question.title);
+    const docRef = doc(db, "questions", questionId);
     await setDoc(docRef, questionDoc);
     for (let i = 0; i < question.examples.length; i++) {
       const add = setDoc(doc(docRef, "examples", (i+1).toString()), question.examples[i]);
@@ -42,4 +42,18 @@ export async function addQuestion(question: Question): Promise<Question> {
   } catch (error) {
     return Promise.reject(error);
   }
-}
+}
+
+export async function addQuestion(question: Question): Promise<Question> {
+  try {
+    let questionDoc: Omit<Question, "examples"> = question
+    const docRef = await addDoc(collection(db, "questions"), questionDoc);
+    for (let i = 0; i < question.examples.length; i++) {
+      const add = setDoc(doc(docRef, "examples", (i+1).toString()), question.examples[i]);
+    }
+    return Promise.resolve(question);
+  } catch (error) {
+    return Promise.reject(error);
+  }
+}
+

frontend/src/api/questions/data.ts

@@ -16,5 +16,8 @@ export const getAllQuestions = () => questionhttpClient.get("/questions");
 export const addQuestion = (params: QuestionDTO) =>
   questionhttpClient.post("/questions", params);
 
+export const updateQuestion = (questionId: string, params: QuestionDTO) =>
+  questionhttpClient.put(`/questions/${questionId}`, params);
+
 export const deleteQuestion = (questionId: string) =>
   questionhttpClient.delete(`/questions/${questionId}`);

frontend/src/components/Questions/EditQuestionTab.tsx

@@ -20,7 +20,7 @@ const EditQuestionTab: React.FC<EditQuestionPreviewProps> = ({question, onEdit,
     }
 
     const handleFinishEdit = (editedQuestion: Question) => {
-        onEdit(question);
+        onEdit(editedQuestion);
         setEditQuestion(false);
     };
 

frontend/src/components/Questions/EditQuestionsTab.tsx

@@ -4,7 +4,7 @@ import {Box, Button} from "@mui/material";
 import Grid from "@mui/material/Grid";
 import Question from "./Question";
 import {Delete} from "@mui/icons-material";
-import {addQuestion, deleteQuestion} from "../../api/questions/data";
+import {addQuestion, deleteQuestion, updateQuestion} from "../../api/questions/data";
 import {AxiosError} from "axios";
 
 
@@ -18,10 +18,20 @@ const EditQuestionsTab: React.FC = () => {
         setEditQuestions(false);
     };
 
-    const onEdit = (editedQuestion: Question) => {
+    const onEdit = async (editedQuestion: Question) => {
         console.log("Edited Question: ", editedQuestion);
 
-        //TODO: Update question in database
+        try {
+            await updateQuestion(editedQuestion.id, editedQuestion);
+            console.log(`Question updated: ${editedQuestion.id}: ${editedQuestion.title}`);
+            setEditQuestions(false);
+        } catch (e) {
+            if (e instanceof AxiosError && e.response) {
+                console.log(e.response.data.code);
+            } else if (e instanceof Error) {
+                console.log(e.message);
+            }
+        }
     }
 
     const onDelete = async (questionToDelete: Question) => {