From 1f37023e7560dea736a17d41fa58520b108235f0 Mon Sep 17 00:00:00 2001
From: Carifio24 <carifio24@gmail.com>
Date: Thu, 12 Sep 2024 12:12:41 -0400
Subject: [PATCH] Combine handlers for redundant endpoints.

---
 src/server.ts | 64 ++++++++-------------------------------------------
 1 file changed, 9 insertions(+), 55 deletions(-)

diff --git a/src/server.ts b/src/server.ts
index 948e56b..e96fcc5 100644
--- a/src/server.ts
+++ b/src/server.ts
@@ -226,7 +226,10 @@ app.listen(PORT, () => {
 });
 
 // Educator sign-up
-app.post("/educator-sign-up", async (req, res) => {
+app.post([
+  "/educators/create",
+  "/educator-sign-up", // Old
+], async (req, res) => {
   const data = req.body;
   const valid = (
     typeof data.firstName === "string" &&
@@ -254,13 +257,16 @@ app.post("/educator-sign-up", async (req, res) => {
 });
 
 // Student sign-up
-app.post("/student-sign-up", async (req, res) => {
+app.post([
+  "/students/create",
+  "/student-sign-up", // Old
+], async (req, res) => {
   const data = req.body;
   const valid = (
     typeof data.username === "string" &&
     typeof data.password === "string" &&
     ((typeof data.institution === "string") || (data.institution == null)) &&
-    typeof data.email === "string" &&
+    ((typeof data.email === "string") || (data.email == null)) &&
     ((typeof data.age === "number") || (data.age == null)) &&
     ((typeof data.gender === "string") || (data.gender == null)) &&
     ((typeof data.classroom_code === "string") || (data.classroom_code == null))
@@ -560,58 +566,6 @@ app.post("/classes/join", async (req, res) => {
   res.json({ success, message });
 });
 
-app.post("/educators/create", async (req, res) => {
-  const data = req.body;
-  const valid = (
-    typeof data.first_name === "string" &&
-    typeof data.last_name === "string" &&
-    typeof data.password === "string" &&
-    ((typeof data.institution === "string") || (data.institution == null)) &&
-    typeof data.email === "string" &&
-    ((typeof data.age === "number") || (data.age == null)) &&
-    ((typeof data.gender === "string") || data.gender == null)
-  );
-
-  let result: SignUpResult;
-  if (valid) {
-    result = await signUpEducator(data);
-  } else {
-    result = SignUpResult.BadRequest;
-    res.status(400);
-  }
-  res.json({
-    educator_info: data,
-    status: result,
-    success: SignUpResult.success(result)
-  });
-});
-
-app.post("/students/create", async (req, res) => {
-  const data = req.body;
-  const valid = (
-    typeof data.username === "string" &&
-    typeof data.password === "string" &&
-    ((typeof data.institution === "string") || (data.institution == null)) &&
-    ((typeof data.email === "string") || (data.email == null)) &&
-    ((typeof data.age === "number") || (data.age == null)) &&
-    ((typeof data.gender === "string") || (data.gender == null)) &&
-    ((typeof data.classroom_code === "string") || (data.classroom_code == null))
-  );
-
-  let result: SignUpResult;
-  if (valid) {
-    result = await signUpStudent(data);
-  } else {
-    result = SignUpResult.BadRequest;
-    res.status(400);
-  }
-  res.json({
-    student_info: data,
-    status: result,
-    success: SignUpResult.success(result)
-  });
-});
-
 /* Classes */
 app.post("/classes/create", async (req, res) => {
   const data = req.body;