adding in proper creation of user

packages/api/routes/user.ts

@@ -1,8 +1,10 @@
 import { GoogleUserInfo, User } from "@nirvana/core/models";
 import express, { Application, Request, Response } from "express";
 
+import { ObjectID } from "bson";
 import { ObjectId } from "mongodb";
 import { UserService } from "../services/user.service";
+import { UserStatus } from "../../core/models/user.model";
 import { authCheck } from "../middleware/auth";
 import { collections } from "../services/database.service";
 
@@ -14,8 +16,6 @@ export default function getUserRoutes() {
   // get user details based on id token
   router.get("/", authCheck, getUserDetails);
 
-  router.post("/create", createUser);
-
   return router;
 }
 
@@ -25,15 +25,18 @@ export default function getUserRoutes() {
  */
 async function getUserDetails(req: Request, res: Response) {
   const email: string = res.locals.email;
+  const userId: string = res.locals.userId;
 
   // passed in accesstoken no matter what
   const { access_token } = req.query;
 
-  console.log(`getting data for ${email}`);
+  console.log(res.locals);
+
+  console.log(`getting data for ${userId}`);
 
   try {
     // return user details if it passed auth middleware
-    const user = await UserService.getUserByEmail(email);
+    const user = await UserService.getUserById(userId);
 
     // if no user found, then go ahead and create user
     if (!user) {
@@ -50,18 +53,21 @@ async function getUserDetails(req: Request, res: Response) {
 
       // create initial user model object
       const newUser = new User(
+        userId,
         userInfo.email,
         userInfo.verifiedEmail,
         userInfo.name,
         userInfo.given_name,
         userInfo.family_name,
         userInfo.picture,
-        userInfo.locale
+        userInfo.locale,
+        new Date(),
+        UserStatus.ONLINE,
+        new Date()
       );
 
       // create user if not exists
       const insertResult = await UserService.createUserIfNotExists(newUser);
-      newUser._id = insertResult?.insertedId;
 
       insertResult
         ? res.status(200).send(newUser)
@@ -72,48 +78,8 @@ async function getUserDetails(req: Request, res: Response) {
 
     // otherwise, just return the user details
     res.status(200).send(user);
-  } catch (error) {
-    res
-      .status(404)
-      .send(`unable to find a matching document with email: ${email}`);
-  }
-}
-
-/** DEPRECATED...USING THE SAME SIGN IN ROUTE TO CREATE */
-async function createUser(req: Request, res: Response) {
-  try {
-    const { access_token } = req.query;
-
-    if (!access_token) {
-      res.status(400).send("No access token provided");
-      return;
-    }
-
-    // get google user info from access token
-    const userInfo: GoogleUserInfo =
-      await UserService.getGoogleUserInfoWithAccessToken(
-        access_token as string
-      );
-
-    // create initial user model object
-    const newUser = new User(
-      userInfo.email,
-      userInfo.verifiedEmail,
-      userInfo.name,
-      userInfo.given_name,
-      userInfo.family_name,
-      userInfo.picture,
-      userInfo.locale
-    );
-
-    // create user if not exists
-    const insertResult = await UserService.createUserIfNotExists(newUser);
-
-    insertResult
-      ? res.status(200).send("User created")
-      : res.status(500).send("Failed to create account, already exists");
   } catch (error) {
     console.log(error);
-    res.status(500).send("Problem in creating user");
+    res.status(500).send(`Problem with signing user up or logging in`);
   }
 }