Uploaded image for project: 'Document Management System'
  1. Document Management System
  2. DMS-1336

Text | Uploaded image not reflected on review and sign

    Details

    • Type: Bug
    • Status: Done
    • Priority: Medium
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: Enhancements - Fall
    • Labels:
      None
    • Company:
      All Clients/Multiple Clients

      Description

      File Type: Text
      1.Create the document template with file type TEXT, upload the image in the body
      2. Send the document
      3. In Email, we are not getting the images added in document template with file type TEXT

      PFA -

        Attachments

        1. AFFTECTED_FILES_DEV_1336.txt
          0.2 kB
        2. image_3g.png
          image_3g.png
          10 kB
        3. Screenshot (3479).png
          Screenshot (3479).png
          113 kB
        4. Screenshot (3480).png
          Screenshot (3480).png
          139 kB
        5. Screenshot (3481).png
          Screenshot (3481).png
          133 kB
        6. Screenshot (3482).png
          Screenshot (3482).png
          131 kB
        7. TextPreview.jpg
          TextPreview.jpg
          254 kB
        8. TextReviewAndSign.jpg
          TextReviewAndSign.jpg
          71 kB

          Activity

          Hide
          mohd.belal Mohd Belal (Inactive) added a comment - - edited

          In Review and Sign page the images are getting populated, but in email we are not getting the images added in Text document.

          Show
          mohd.belal Mohd Belal (Inactive) added a comment - - edited In Review and Sign page the images are getting populated, but in email we are not getting the images added in Text document.
          Hide
          akash.thakur Akash Thakur (Inactive) added a comment - - edited

          Hi All,

          Please Find below 3'Cs for this Jira

          Concern:-  In Email we are not getting the images added in document template with file type TEXT.

          Cause:-  For Template type TEXT we are using CKEditor for HTML body Creation. & it uses Base64 encoding by default to embed images inserted in the body which is almost blocked on email servers for security purpose thus images are not displayed in the received email.

          Correction:- We are converting Base64 encoded string used as a source in image tags from the HTML body of template to the actual image and writing them on fileserver once this is done we move these images from file server to web server and replace the respective image source with the total absolute path of the image  copied to web server folder.

          Rohan J Khandave please request you to review the code.

          PFA:- Affected Files & image_3g.png for reference.

           

          Thanks,

          Akash Thakur

          Swapnil Pandhare Rohan J Khandave Samir

          Show
          akash.thakur Akash Thakur (Inactive) added a comment - - edited Hi All, Please Find below 3'Cs for this Jira Concern :-  In Email we are not getting the images added in document template with file type TEXT. Cause :-  For Template type TEXT we are using CKEditor for HTML body Creation. & it uses Base64 encoding by default to embed images inserted in the body which is almost blocked on email servers for security purpose thus images are not displayed in the received email. Correction :- We are converting Base64 encoded string used as a source in image tags from the HTML body of template to the actual image and writing them on fileserver once this is done we move these images from file server to web server and replace the respective image source with the total absolute path of the image  copied to web server folder. Rohan J Khandave please request you to review the code. PFA:- Affected Files &  image_3g.png  for reference.   Thanks, Akash Thakur Swapnil Pandhare Rohan J Khandave Samir
          Hide
          akash.thakur Akash Thakur (Inactive) added a comment - - edited

          Hi Rohan J Khandave ,

          Please have a review of the changes suggested.

          PR:- https://github.com/cbdr/DMS/pull/620

          Thanks,

          Akash Thakur

           

          Rohan J Khandave

          Show
          akash.thakur Akash Thakur (Inactive) added a comment - - edited Hi Rohan J Khandave , Please have a review of the changes suggested. PR:-  https://github.com/cbdr/DMS/pull/620 Thanks, Akash Thakur   Rohan J Khandave
          Hide
          rohan.khandave Rohan J Khandave (Inactive) added a comment -

          Please review comments from code review.

          Show
          rohan.khandave Rohan J Khandave (Inactive) added a comment - Please review comments from code review.
          Hide
          akash.thakur Akash Thakur (Inactive) added a comment -

          Hi Samir,

          I have completed all the changes suggested by Rohan J Khandave & Harshveer Singh the code is refactored multiple time in order to be optimized. I have written a test case for the changes but it requires session data so once Ganesh Sadawarte commits his structure changes we will update the test case for this bug.

          Rohan J Khandave (Harshveer Singh has reviewed all the changes he suggested). please request you to review the template date related changes & test case wrote.

          Thanks,
          Akash Thakur
          Ganesh Sadawarte

          Show
          akash.thakur Akash Thakur (Inactive) added a comment - Hi Samir , I have completed all the changes suggested by Rohan J Khandave & Harshveer Singh the code is refactored multiple time in order to be optimized. I have written a test case for the changes but it requires session data so once Ganesh Sadawarte commits his structure changes we will update the test case for this bug. Rohan J Khandave ( Harshveer Singh has reviewed all the changes he suggested). please request you to review the template date related changes & test case wrote. Thanks, Akash Thakur Ganesh Sadawarte
          Hide
          akash.thakur Akash Thakur (Inactive) added a comment -

          Hi All,

          This branch will be merged after DMS-1342 changes.it was pending for test case structure changes done inside 1342 branch.

          performing a test to make it ready to merge into develop.

          Thanks,

          Akash Thakur

          Samir Narendra Kumar Harshveer Singh Rohan J Khandave Ganesh Sadawarte

          Show
          akash.thakur Akash Thakur (Inactive) added a comment - Hi All, This branch will be merged after DMS-1342 changes.it was pending for test case structure changes done inside 1342 branch. performing a test to make it ready to merge into develop. Thanks, Akash Thakur Samir Narendra Kumar Harshveer Singh Rohan J Khandave Ganesh Sadawarte
          Hide
          priya.dhamande Priya Dhamande (Inactive) added a comment -

          Environment: Production
          Login: Admin and Candidate
          Browser: FF and Chrome
          Page: Review and Sign
          File Type: text

          Image in Text File Type

          • Can be seen on Preview of Admin
          • Review and Sign of Candidate
          • Able to Submit text Image document
          • Able to Decline text Image document

          Refer screen capture. So, marking jira done.

          Sachin Hingole Hrishikesh Deshpande Samir Rohan J Khandave

          Refer screen capture. So, marking jira done.

          Show
          priya.dhamande Priya Dhamande (Inactive) added a comment - Environment: Production Login: Admin and Candidate Browser: FF and Chrome Page: Review and Sign File Type: text Image in Text File Type Can be seen on Preview of Admin Review and Sign of Candidate Able to Submit text Image document Able to Decline text Image document Refer screen capture. So, marking jira done. Sachin Hingole Hrishikesh Deshpande Samir Rohan J Khandave Refer screen capture. So, marking jira done.

            People

            Assignee:
            akash.thakur Akash Thakur (Inactive)
            Reporter:
            jayshree.nagpure Jayshree Nagpure (Inactive)
            Developer:
            Akash Thakur (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 24h Original Estimate - 24h
                24h
                Remaining:
                Time Spent - 28h 20m Remaining Estimate - 29.5h
                29.5h
                Logged:
                Time Spent - 28h 20m Remaining Estimate - 29.5h
                28h 20m