Merge pull request #130 from manipalutsav/testing

Author: abhikpai

src/app.js

@@ -71,6 +71,7 @@ const practiceSlot = require("./routes/practiceSlot");
 const notifications = require("./routes/notifications");
 const settings = require("./routes/settings");
 const slotting = require("./routes/slotting");
+const feedback = require("./routes/feedback");
 
 app.use("/colleges", collegesRouter);
 app.use("/events", eventsRouter);
@@ -88,7 +89,7 @@ app.use("/practiceSlots", practiceSlot);
 app.use("/notifications", notifications);
 app.use("/settings", settings);
 app.use("/slotting", slotting);
-
+app.use("/feedback", feedback)
 
 // Error handlers
 app.use(handle404);

src/controllers/feedback.js

@@ -0,0 +1,60 @@
+const Feedback = require("../models/Feedback");
+
+exports.create = async(req, res) => {
+    try{
+        const {rating, comment, signature, judge, event} = req.body;
+        if(!rating || !comment, !signature || !judge || !event){
+            return res.status(400).json({
+                status: 400,
+                message: "Rating, comment, signature, judge id and event id are required ⁉️",
+            });
+        }
+        const feedback = await Feedback({
+            rating,
+            comment,
+            signature,
+            judge,
+            event
+        })
+        const result = await feedback.save();
+        return res.status(200).json({
+            status: 200,
+            message: "Success",
+            data: result
+        });
+    }catch(e){
+        return res.status(501).json({
+            status: 501,
+            message: "Internal Server Error ❌",
+        });
+    }
+}
+
+exports.getFeedbackForAnEvent = async(req, res) => {
+    try{
+        const {event} = req.body;
+        if(!event){
+            return res.status(400).json({
+                status: 400,
+                message: "Event id is required ⁉️",
+            });
+        }
+        const feedback = await Feedback.find({ event }).populate("judge", "name");
+        if(!feedback){
+            return res.status(404).json({
+                status: 404,
+                message: "No feedback found for the event 🚫",
+            });
+        }
+        return res.status(200).json({
+            status: 200,
+            message: "Success",
+            data: feedback
+        });
+    }catch(e){
+        return res.status(501).json({
+            status: 501,
+            message: "Internal Server Error ❌",
+        });
+    }
+}

src/controllers/judges.js

@@ -1,8 +1,8 @@
 "use strict";
-
+const ObjectId = require("mongodb").ObjectId;
 const JudgeModel = require("../models/Judge");
 const JudgeScoreModel = require("../models/JudgeScore");
-
+const FeedbackModel = require("../models/Feedback");
 const getAll = async (req, res) => {
   try {
     let judges = await JudgeModel.find();
@@ -89,16 +89,17 @@ const create = async (req, res) => {
   }
 };
 /**
- * Deletes a judge by ID and removes all associated judge score records.
+ * Deletes a judge by ID and removes all associated judge score records and feedback.
  * 
  * This function:
  * 1. Extracts the judge ID from the request parameters.
  * 2. Checks if the judge exists in the database.
  * 3. If the judge does not exist, returns a 404 error response.
  * 4. Deletes all `JudgeScore` records linked to the specified judge.
- * 5. Deletes the judge record from the database.
- * 6. Returns a success response if the operation is completed successfully.
- * 7. Handles any errors by returning a 500 Internal Server Error response.
+ * 5. Deletes all `Feedback` records linked to the specified judge.
+ * 6. Deletes the judge record from the database.
+ * 7. Returns a success response if the operation is completed successfully.
+ * 8. Handles any errors by returning a 500 Internal Server Error response.
  * 
  * @param {Object} req - Express request object containing the judge ID in `req.params.id`.
  * @param {Object} res - Express response object for sending JSON responses.
@@ -122,6 +123,7 @@ const deleteJudgeById = async (req, res) => {
 
     // Delete the judge
     await JudgeModel.findByIdAndDelete(judge_id);
+    await FeedbackModel.deleteOne({judge: ObjectId((judge_id))});
 
     return res.json({
       status: 200,

src/controllers/settings.js

@@ -32,7 +32,6 @@ const get = async (req, res) => {
         data: settings[setting],
       });
     }
-
     return res.json({
       status: 404,
       message: "No setting found with the given name",
@@ -70,8 +69,9 @@ const getAll = async (req, res) => {
 };
 
 const updateSettings = async (req, res) => {
-  let { title, editTeamEnabled } = req.body;
-  if( title === null || editTeamEnabled === null) {
+  
+  let { title, editTeamEnabled, downloadCertificateEnabled } = req.body;
+  if( title === null || editTeamEnabled === null || downloadCertificateEnabled === null) {
     return res.status(404).json({
       status: 400,
       message: "Bad request",
@@ -85,10 +85,12 @@ const updateSettings = async (req, res) => {
     setting = await SettingModel.create({
       title: title,
       editTeamEnabled: editTeamEnabled,
+      downloadCertificateEnabled: downloadCertificateEnabled,
     });
   }else{
     setting.title = title;
     setting.editTeamEnabled = editTeamEnabled;
+    setting.downloadCertificateEnabled = downloadCertificateEnabled;
     await setting.save();
   }
 

src/models/Feedback.js

@@ -0,0 +1,33 @@
+const mongoose = require("mongoose");
+
+const schema = {
+    rating:{
+        type: Number,
+        required: true
+    },
+    comment: {
+        type: String
+    },
+    signature: {
+        type: String,
+        required: true
+    },
+    judge: {
+        type: mongoose.Schema.Types.ObjectId,
+        ref: "Judge",
+        required: true
+    },
+    event: {
+        type: mongoose.Schema.Types.ObjectId,
+        ref: "Event",
+        required: true
+    }
+};
+
+const options = {
+    autoCreate: true,
+};
+
+const feedbackSchema = new mongoose.Schema(schema, options);
+
+module.exports = mongoose.model("Feedback", feedbackSchema);

src/models/Setting.js

@@ -10,6 +10,10 @@ const schema = {
     type: Boolean,
     required: true,
   },
+  downloadCertificateEnabled:{
+    type: Boolean,
+    required: true,
+  }
 };
 
 const options = {

src/routes/feedback.js

@@ -0,0 +1,7 @@
+const router = require("express").Router();
+const { create, getFeedbackForAnEvent } = require("../controllers/feedback");
+
+router.post("/", create);
+router.post("/getFeedback", getFeedbackForAnEvent);
+
+module.exports = router;