Merge pull request #1492 from OpenSignLabs/fix_uncaught

OpenSignLabs · Nov 28, 2024 · 0275762 · 0275762
2 parents 37b41fc + 618b4fa
commit 0275762
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/apps/OpenSign/src/pages/PdfRequestFiles.js b/apps/OpenSign/src/pages/PdfRequestFiles.js
@@ -531,9 +531,9 @@ function PdfRequestFiles(props) {
           const obj = documentData?.[0];
           setSendInOrder(obj?.SendinOrder || false);
           if (
-            obj &&
-            obj?.Signers?.length > 0 &&
-            obj?.Placeholders?.length > 0
+            obj?.Signers?.length &&
+            obj?.Placeholders?.length &&
+            currUserId !== "undefined"
           ) {
             const params = {
               event: "viewed",

diff --git a/apps/OpenSign/src/pages/PlaceHolderSign.js b/apps/OpenSign/src/pages/PlaceHolderSign.js
@@ -1599,7 +1599,11 @@ function PlaceHolderSign() {
     }
   };
   const handleRecipientSign = () => {
-    navigate(`/recipientSignPdf/${documentId}/${currentId}`);
+    if (currentId) {
+      navigate(`/recipientSignPdf/${documentId}/${currentId}`);
+    } else {
+      navigate(`/recipientSignPdf/${documentId}`);
+    }
   };
   const handleLinkUser = (id) => {
     setIsAddUser({ [id]: true });