fix: avoid creating document meta with empty strings

apps/web/src/components/forms/edit-document/add-subject.action.ts

@@ -15,12 +15,18 @@ export const completeDocument = async ({ documentId, email }: CompleteDocumentAc
 
   const { id: userId } = await getRequiredServerComponentSession();
 
+  if (!email.message && !email.subject) {
+    return await sendDocument({
+      userId,
+      documentId,
+    });
+  }
+
   const createDocumentMetaResponse = await createDocumentMeta({
     emailBody: email.message,
     emailSubject: email.subject,
   });
 
-  if (createDocumentMetaResponse) {
   await updateDocument({
     documentId,
     data: {
@@ -31,7 +37,6 @@ export const completeDocument = async ({ documentId, email }: CompleteDocumentAc
       },
     },
   });
-  }
 
   await sendDocument({
     userId,

packages/lib/server-only/document-meta/create-document-meta.ts

@@ -11,26 +11,23 @@ export const createDocumentMeta = async ({
   emailBody,
   emailSubject,
 }: CreateDocumentMetaOptions) => {
-  const documentMeta = await prisma.documentMeta.findFirst();
+  const emailData = {
-
-  if (!documentMeta) {
-    return await prisma.documentMeta.create({
-      data: {
     customEmailBody: emailBody,
     customEmailSubject: emailSubject,
-      },
+  };
+
+  const existingDocumentMeta = await prisma.documentMeta.findFirst({
+    where: emailData,
   });
-  }
 
-  if (emailBody && emailSubject) {
+  if (existingDocumentMeta) {
     return await prisma.documentMeta.update({
-      where: {
+      where: { id: existingDocumentMeta.id },
-        id: documentMeta.id,
+      data: emailData,
-      },
+    });
-      data: {
+  } else {
-        customEmailBody: emailBody,
+    return await prisma.documentMeta.create({
-        customEmailSubject: emailSubject,
+      data: emailData,
-      },
     });
   }
 };