Merge pull request #1282 from OpenSignLabs/form-layout

fix: document download issue in the disabled OTP public sign flow before signing and subscription issue in the guest login process

Author: prafull-opensignlabs

apps/OpenSign/public/locales/en/translation.json

@@ -265,10 +265,10 @@
   "make-template-public-alert": "Are you sure you want to make this template public?",
   "make-template-private-alert": "Are you sure you want to make this template private? This will remove it from your public profile.",
   "public-role": "Public role",
-  "public-url": "Public URL",
+  "public-url": "Public profile needed",
   "public-url-copy": "Here’s your public URL: ",
   "public-url-copy-mssg":"Copy it or share it with the signer, and you will be able to see all your publicly set templates.",
-  "add-public-url-alert": "Please add your public URL, and you will be able to make a public template.",
+  "add-public-url-alert": "Please complete your OpenSign public profile setup. Once done, you’ll be able to make templates public. This process only needs to be completed once.",
   "share-with-alert": "You cannot share a template if any roles already have contacts assigned. Please remove all contact assignments from the roles before sharing the template.",
   "share-with": "Share with",
   "team-not-found": "Team not found",
@@ -672,6 +672,7 @@
   "document-logs":"Document logs",
   "server-down": "Unable to connect to the OpenSign server. If you are self-hosting OpenSign, please ensure that all the steps in the documentation have been followed correctly. If you're running OpenSign locally, you might be accessing it through an incorrect port number.",
   "admin-exists": "Admin already exists. Please login to the application using admin credentials in order to manage users.",
-   "public-tour-message":"Please make template public to copy public URL"
+  "public-tour-message":"The template needs to be public before you can generate a shareable link.",
+  "add-user-template": "You need to add a role before you can add fields for it."
 
 }

apps/OpenSign/public/locales/fr/translation.json

@@ -264,10 +264,10 @@
   "make-template-public-alert": "Êtes-vous sûr de vouloir rendre ce modèle public ?",
   "make-template-private-alert": "Êtes-vous sûr de vouloir rendre ce modèle privé ? Cela le supprimera de votre profil public.",
   "public-role": "Rôle public",
-  "public-url": "URL publique",
+  "public-url": "Profil public requis",
   "public-url-copy": "Voici votre URL publique : ",
   "public-url-copy-mssg": "copiez-la ou partagez-le avec le signataire et vous pouvoir voir tous vos paramètres publics modèles.",
-  "add-public-url-alert": "Veuillez ajouter votre URL publique et vous pourra rendre public modèle.",
+  "add-public-url-alert": "Veuillez terminer la configuration de votre profil public OpenSign. Une fois cela fait, vous pourrez rendre les modèles publics. Ce processus ne doit être effectué qu’une seule fois.",
   "share-with-alert": "Vous ne pouvez pas partager un modèle si des contacts sont déjà attribués à des rôles. Veuillez supprimer toutes les attributions de contacts des rôles avant de partager le modèle.",
   "share-with": "Partager avec",
   "team-not-found": "Équipe introuvable",
@@ -671,7 +671,8 @@
   "document-logs":"Journaux de documents",
   "server-down": "Impossible de se connecter au serveur OpenSign. Si vous hébergez vous-même OpenSign, veuillez vous assurer que toutes les étapes de la documentation ont été correctement suivies. Si vous exécutez OpenSign localement, vous y accédez peut-être via un numéro de port incorrect.",
   "admin-exists": "L'administrateur existe déjà. Veuillez vous connecter à l'application en utilisant les informations d'identification de l'administrateur afin de gérer les utilisateurs.",
-  "public-tour-message":"Veuillez rendre le modèle public pour copier l'URL publique"
+  "public-tour-message": "Le modèle doit être public avant que vous puissiez générer un lien partageable.",
+  "add-user-template":"Vous devez ajouter un rôle avant de pouvoir lui ajouter des champs. "
 
 
 }

apps/OpenSign/src/components/pdf/PdfHeader.js

@@ -418,11 +418,7 @@ function Header({
                     {templateId && (
                       <button
                         onClick={() =>
-                          handleDownloadPdf(
-                            pdfDetails,
-                            pdfUrl,
-                            setIsDownloading
-                          )
+                          handleDownloadPdf(pdfDetails, setIsDownloading)
                         }
                         type="button"
                         className="op-btn op-btn-ghost op-btn-sm mr-[3px]"

apps/OpenSign/src/pages/PdfRequestFiles.js

@@ -73,7 +73,7 @@ function PdfRequestFiles(props) {
   const [selectWidgetId, setSelectWidgetId] = useState("");
   const [otpLoader, setOtpLoader] = useState(false);
   const [isCelebration, setIsCelebration] = useState(false);
-  const [requestSignTour, setRequestSignTour] = useState(false);
+  const [requestSignTour, setRequestSignTour] = useState(true);
   const [tourStatus, setTourStatus] = useState([]);
   const [isLoading, setIsLoading] = useState({
     isLoad: true,
@@ -129,7 +129,7 @@ function PdfRequestFiles(props) {
   const [contact, setContact] = useState({ name: "", phone: "", email: "" });
   const [isOtp, setIsOtp] = useState(false);
   const [loading, setLoading] = useState(false);
-  const [res, setRes] = useState({});
+  const [publicRes, setPublicRes] = useState({});
   const [documentId, setDocumentId] = useState("");
   const [isPublicContact, setIsPublicContact] = useState(false);
   const [pdfArrayBuffer, setPdfArrayBuffer] = useState("");
@@ -429,7 +429,7 @@ function PdfRequestFiles(props) {
 
         currUserId = getCurrentSigner?.objectId
           ? getCurrentSigner.objectId
-          : signerObjectId || "";
+          : contactBookId || signerObjectId || "";
         if (isEnableSubscription) {
           await checkIsSubscribed(
             documentData[0]?.ExtUserPtr?.objectId,
@@ -1510,16 +1510,14 @@ function PdfRequestFiles(props) {
       );
 
       if (userRes?.data?.result) {
-        setRes(userRes.data.result);
+        setPublicRes(userRes.data.result);
         const isEnableOTP = pdfDetails?.[0]?.IsEnableOTP || false;
         if (isEnableOTP) {
           await SendOtp();
         } else {
           setIsPublicContact(false);
           setIsPublicTemplate(false);
-          setLoading(false);
           setDocumentId(userRes.data?.result?.docId);
-          getDocumentDetails(userRes?.data?.result?.docId);
           const contactId = userRes.data.result?.contactId;
           setSignerObjectId(contactId);
         }
@@ -1564,7 +1562,7 @@ function PdfRequestFiles(props) {
 
   const SendOtp = async () => {
     try {
-      const params = { email: contact.email, docId: res.docId };
+      const params = { email: contact.email, docId: publicRes?.docId };
       const Otp = await axios.post(
         `${localStorage.getItem("baseUrl")}/functions/SendOTPMailV1`,
         params,
@@ -1630,17 +1628,13 @@ function PdfRequestFiles(props) {
               JSON.stringify(contractUserDetails)
             );
           }
-
           localStorage.setItem("username", _user.name);
           localStorage.setItem("accesstoken", _user.sessionToken);
           setLoading(false);
-          // navigate(`/load/recipientSignPdf/${res?.docId}/${res?.contactId}`);
-          // document.getElementById("my_modal").close();
           setIsPublicContact(false);
           setIsPublicTemplate(false);
           setIsLoading({ isLoad: false });
-          setDocumentId(res?.docId);
-          getDocumentDetails(res?.docId);
+          setDocumentId(publicRes?.docId);
         }
       } catch (error) {
         console.log("err ", error);

apps/OpenSign/src/pages/TemplatePlaceholder.js

@@ -508,7 +508,7 @@ const TemplatePlaceholder = () => {
   const tourAddRole = [
     {
       selector: '[data-tut="reactourAddbtn"]',
-      content: "You need to add a role before you can add fields for it. ",
+      content: t("add-user-template"),
       position: "top",
       style: { fontSize: "13px" }
     }

apps/OpenSign/src/pages/UserProfile.js

@@ -611,7 +611,7 @@ function UserProfile() {
                     <div className="flex flex-row md:items-center gap-1">
                       {editmode || !extendUser?.[0]?.Tagline ? (
                         <input
-                          maxLength={40}
+                          maxLength={100}
                           onChange={handleOnchangeTagLine}
                           value={tagLine}
                           disabled={!editmode}

apps/OpenSign/src/primitives/GetReportDisplay.js

@@ -1511,14 +1511,14 @@ const ReportTable = (props) => {
                                         <span>{t("public-url-copy")}</span>
                                         <div className=" flex items-center justify-between gap-3 p-[2px] ">
                                           <div className="w-[280px] whitespace-nowrap overflow-hidden text-ellipsis">
-                                            <span
-                                              onClick={() =>
-                                                copytoProfileLink()
-                                              }
+                                            <a
+                                              rel="noreferrer"
+                                              target="_blank"
+                                              href={publicUrl()}
                                               className="ml-[2px] underline underline-offset-2 cursor-pointer text-blue-800"
                                             >
                                               {publicUrl()}
-                                            </span>
+                                            </a>
                                           </div>
                                           <div className="flex items-center gap-2">
                                             <RWebShare
@@ -1535,7 +1535,7 @@ const ReportTable = (props) => {
                                               </button>
                                             </RWebShare>
                                             <button
-                                              className="op-btn op-btn-primary op-btn-outline op-btn-xs md:op-btn-sm"
+                                              className="op-btn op-btn-primary op-btn-outline op-btn-xs md:op-btn-sm md:w-[100px]"
                                               onClick={() =>
                                                 copytoProfileLink()
                                               }
@@ -1561,7 +1561,7 @@ const ReportTable = (props) => {
                                           className="mt-3 op-btn op-btn-primary"
                                           onClick={() => navigate("/profile")}
                                         >
-                                          {t("add")}
+                                          {t("Proceed")}
                                         </button>
                                       </div>
                                     )}

apps/OpenSign/src/script/locales/en/translation.json

@@ -265,10 +265,10 @@
   "make-template-public-alert": "Are you sure you want to make this template public?",
   "make-template-private-alert": "Are you sure you want to make this template private? This will remove it from your public profile.",
   "public-role": "Public role",
-  "public-url": "Public URL",
+  "public-url": "Public profile needed",
   "public-url-copy": "Here’s your public URL: ",
   "public-url-copy-mssg": "Copy it or share it with the signer, and you will be able to see all your publicly set templates.",
-  "add-public-url-alert": "Please add your public URL, and you will be able to make a public template.",
+  "add-public-url-alert": "Please complete your OpenSign public profile setup. Once done, you’ll be able to make templates public. This process only needs to be completed once.",
   "share-with-alert": "You cannot share a template if any roles already have contacts assigned. Please remove all contact assignments from the roles before sharing the template.",
   "share-with": "Share with",
   "team-not-found": "Team not found",
@@ -672,5 +672,6 @@
   "document-logs": "Document logs",
   "server-down": "Unable to connect to the OpenSign server. If you are self-hosting OpenSign, please ensure that all the steps in the documentation have been followed correctly. If you're running OpenSign locally, you might be accessing it through an incorrect port number.",
   "admin-exists": "Admin already exists. Please login to the application using admin credentials in order to manage users.",
-  "public-tour-message": "Please make template public to copy public URL"
+  "public-tour-message":"The template needs to be public before you can generate a shareable link.",
+  "add-user-template": "You need to add a role before you can add fields for it."
 }

apps/OpenSign/src/script/locales/fr/translation.json

@@ -264,10 +264,10 @@
   "make-template-public-alert": "Êtes-vous sûr de vouloir rendre ce modèle public ?",
   "make-template-private-alert": "Êtes-vous sûr de vouloir rendre ce modèle privé ? Cela le supprimera de votre profil public.",
   "public-role": "Rôle public",
-  "public-url": "URL publique",
+  "public-url": "Profil public requis",
   "public-url-copy": "Voici votre URL publique : ",
   "public-url-copy-mssg": "copiez-la ou partagez-le avec le signataire et vous pouvoir voir tous vos paramètres publics modèles.",
-  "add-public-url-alert": "Veuillez ajouter votre URL publique et vous pourra rendre public modèle.",
+  "add-public-url-alert": "Veuillez terminer la configuration de votre profil public OpenSign. Une fois cela fait, vous pourrez rendre les modèles publics. Ce processus ne doit être effectué qu’une seule fois.",
   "share-with-alert": "Vous ne pouvez pas partager un modèle si des contacts sont déjà attribués à des rôles. Veuillez supprimer toutes les attributions de contacts des rôles avant de partager le modèle.",
   "share-with": "Partager avec",
   "team-not-found": "Équipe introuvable",
@@ -671,5 +671,6 @@
   "document-logs": "Journaux de documents",
   "server-down": "Impossible de se connecter au serveur OpenSign. Si vous hébergez vous-même OpenSign, veuillez vous assurer que toutes les étapes de la documentation ont été correctement suivies. Si vous exécutez OpenSign localement, vous y accédez peut-être via un numéro de port incorrect.",
   "admin-exists": "L'administrateur existe déjà. Veuillez vous connecter à l'application en utilisant les informations d'identification de l'administrateur afin de gérer les utilisateurs.",
-  "public-tour-message": "Veuillez rendre le modèle public pour copier l'URL publique"
+  "public-tour-message": "Le modèle doit être public avant que vous puissiez générer un lien partageable.",
+  "add-user-template":"Vous devez ajouter un rôle avant de pouvoir lui ajouter des champs. "
 }