Denormalize signup res and query types

uwblueprint · Mar 18, 2022 · 981f3e9 · 981f3e9
1 parent da88911
commit 981f3e9
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 3 deletions.
diff --git a/backend/graphql/types/shiftSignupType.ts b/backend/graphql/types/shiftSignupType.ts
@@ -22,6 +22,8 @@ const shiftSignupType = gql`
 
   type ShiftSignupResponseDTO {
     shiftId: ID!
+    shiftStartTime: DateTime!
+    shiftEndTime: DateTime!
     userId: ID!
     numVolunteers: Int!
     note: String!

diff --git a/backend/services/implementations/shiftSignupService.ts b/backend/services/implementations/shiftSignupService.ts
@@ -1,5 +1,5 @@
 import { PrismaClient, Signup, SignupStatus } from "@prisma/client";
-import { Prisma } from ".prisma/client";
+import { Prisma, Shift } from ".prisma/client";
 
 import IShiftSignupService from "../interfaces/shiftSignupService";
 import {
@@ -24,6 +24,19 @@ class ShiftSignupService implements IShiftSignupService {
     };
   };
 
+  convertSignupResponseToDTO = (
+    signup: Signup,
+    shift: Shift,
+  ): ShiftSignupResponseDTO => {
+    return {
+      ...signup,
+      shiftId: String(signup.shiftId),
+      userId: String(signup.userId),
+      shiftStartTime: shift.startTime,
+      shiftEndTime: shift.endTime,
+    };
+  };
+
   async getShiftSignupsForUser(
     userId: string,
     signupStatus: SignupStatus | null,
@@ -43,8 +56,13 @@ class ShiftSignupService implements IShiftSignupService {
       };
       const shiftSignups = await prisma.signup.findMany({
         where: filter,
+        include: {
+          shift: true,
+        },
       });
-      return shiftSignups.map((signup) => this.convertSignupToDTO(signup));
+      return shiftSignups.map((signup) =>
+        this.convertSignupResponseToDTO(signup, signup.shift),
+      );
     } catch (error: unknown) {
       Logger.error(
         `Failed to shift signups. Reason = ${getErrorMessage(error)}`,

diff --git a/backend/types.ts b/backend/types.ts
@@ -132,7 +132,10 @@ export type UpdateShiftSignupRequestDTO = Omit<
   "shiftId" | "userId"
 >;
 
-export type ShiftSignupResponseDTO = ShiftSignupDTO;
+export type ShiftSignupResponseDTO = {
+  shiftStartTime: Date;
+  shiftEndTime: Date;
+} & ShiftSignupDTO;
 
 export type SkillDTO = {
   id: string;