From e057dc028b37e9dfa1e76be16402a15dbb212c62 Mon Sep 17 00:00:00 2001
From: lavanyagarg112 <lavanyagarg@u.nus.edu>
Date: Thu, 10 Jul 2025 18:16:11 +0800
Subject: [PATCH] Fix bug and clean code

---
 routes/courses.js    |  2 +-
 routes/onboarding.js |  4 ----
 routes/roadmaps.js   | 16 ----------------
 3 files changed, 1 insertion(+), 21 deletions(-)

diff --git a/routes/courses.js b/routes/courses.js
index 419c898..ca33a7c 100644
--- a/routes/courses.js
+++ b/routes/courses.js
@@ -551,7 +551,7 @@ router.post("/add-module", upload.single("file"), async (req, res) => {
       userId: session.userId,
       organisationId: session.organisation.id,
       action: "add_module",
-      metadata: { moduleId },
+      metadata: { module_id },
       displayMetadata: { "module name": name },
     });
     return res.status(201).json({
diff --git a/routes/onboarding.js b/routes/onboarding.js
index 7ac7b7f..0de8490 100644
--- a/routes/onboarding.js
+++ b/routes/onboarding.js
@@ -479,10 +479,6 @@ router.post("/responses", async (req, res) => {
               [roadmapId, moduleIds[i], i + 1]
             );
           }
-
-          console.log(
-            `Auto-generated roadmap "${roadmapResult.rows[0].name}" with ${moduleIds.length} modules for user ${user.userId}`
-          );
         }
       }
     } catch (roadmapError) {
diff --git a/routes/roadmaps.js b/routes/roadmaps.js
index 584ecde..dc02ab1 100644
--- a/routes/roadmaps.js
+++ b/routes/roadmaps.js
@@ -578,24 +578,8 @@ router.post("/generate", async (req, res) => {
          ORDER BY matching_skills DESC, channel_match DESC, level_match DESC, random_score
          LIMIT 10`; // Limit to top 10 modules
 
-      console.log("Generated Query:", query);
-      console.log("Query Parameters:", params);
-
       const modulesResult = await client.query(query, params);
 
-      console.log("Query Results:", modulesResult.rows.length, "modules found");
-      console.log(
-        "Module Details:",
-        modulesResult.rows.map((row) => ({
-          id: row.id,
-          matching_skills: row.matching_skills,
-          channel_match: row.channel_match,
-          level_match: row.level_match,
-          channel_id: row.channel_id,
-          level_id: row.level_id,
-        }))
-      );
-
       if (modulesResult.rows.length > 0) {
         const moduleIds = modulesResult.rows.map((row) => row.id);