fix: record activity if applicable instead of throwing

Author: axelrindle

apps/api/src/trpc.ts

@@ -57,15 +57,16 @@ const logActivityMiddleware = middleware(async (opts) => {
     if (type !== undefined) {
       // const rawInput = opts.getRawInput() as { id: number }
       const resultData = result.data as { id?: string }
-      logger.verbose(`Recording activity ${opts.path} of type ${type}`)
       const subjectId = type === 'CREATE' ? resultData?.id : '9999'
-      if (!subjectId) throw new TRPCError({ code: 'INTERNAL_SERVER_ERROR', message: 'No subjectId found' })
-      await logActivity({
-        subjectId,
-        subjectType: subject || 'NO_SUBJECT',
-        causerId: opts.ctx.authenticated ? opts.ctx.accountId : undefined,
-        type,
-      })
+      if (subjectId) {
+        logger.verbose(`Recording activity ${opts.path} of type ${type}`)
+        await logActivity({
+          subjectId,
+          subjectType: subject || 'NO_SUBJECT',
+          causerId: opts.ctx.authenticated ? opts.ctx.accountId : undefined,
+          type,
+        })
+      }
     }
   }