Document Management SystemProject Type: software

Document Management System

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

Notifications and Alerts | Invalid Email or special characters accepted

    Details

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

      Description

      Replication steps :
      1) Open the Dashboard
      2) Navigate to email template > Account settings
      3) Enter invalid email or special characters in "Emails to Carbon Copy when any signature requests is sent (Note, this email will not be dispalyed on the create request page)" and
      "Change reply-to address for invite emails (currently goes to the senders email) Carbon Copies" textarea.

      Expected : System should show error message when incorrect email or special characters are inserted

        Attachments

        1. DMS-303.jpg
          DMS-303.jpg
          119 kB
        2. DMS-303BugFix.jpg
          DMS-303BugFix.jpg
          200 kB
        3. NotificationsAlert.jpg
          NotificationsAlert.jpg
          137 kB

          Issue Links

          relates to

          Maintenance - Need to do something like security updates or server maintenance DMS-256 Sprint 4 - DMS Testing

          • Medium - Has the potential to affect progress.
          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            jayshree.nagpure Jayshree Nagpure (Inactive) added a comment -

            cc - Priya Dhamande Samir Swapnil Pandhare Satya

            Show
            jayshree.nagpure Jayshree Nagpure (Inactive) added a comment - cc - Priya Dhamande Samir Swapnil Pandhare Satya
            Hide
            Permalink
            mohd.belal Mohd Belal (Inactive) added a comment -

            Hi, 

            Following check has been added in emails section. Now emails can be entered using a semi-colon as a separator for multiple emails.

             

            The code has been updated on DMS-303.

             

            cc- Samir, Swapnil Pandhare

            Also created a pull request over GitHub for this.

            Thanks.

            Show
            mohd.belal Mohd Belal (Inactive) added a comment - Hi,  Following check has been added in emails section. Now emails can be entered using a semi-colon as a separator for multiple emails.   The code has been updated on DMS-303 .   cc- Samir , Swapnil Pandhare Also created a pull request over GitHub for this. Thanks.
            Hide
            Permalink
            jayshree.nagpure Jayshree Nagpure (Inactive) added a comment -

            Hi Mohd Belal,

            We have verified this issue on preproduction.
            But still we are able to replicate this issue.
            PFA -

            Not working as expected hence changing status as In-progress and assigning jira to you.

            Thanks,
            Jayshree

            cc - Priya Dhamande Samir Swapnil Pandhare

            Show
            jayshree.nagpure Jayshree Nagpure (Inactive) added a comment - Hi Mohd Belal , We have verified this issue on preproduction. But still we are able to replicate this issue. PFA - Not working as expected hence changing status as In-progress and assigning jira to you. Thanks, Jayshree cc - Priya Dhamande Samir Swapnil Pandhare
            Hide
            Permalink
            mohd.belal Mohd Belal (Inactive) added a comment -

            Hi,

            This feature branch is not yet merged with develop branch.

            I have added a New Pull Request for this and added Swapnil Pandhare and Ganesh Sadawarte to review the same.

             

            Thanks.

            Show
            mohd.belal Mohd Belal (Inactive) added a comment - Hi, This feature branch is not yet merged with develop branch. I have added a New Pull Request for this and added Swapnil Pandhare and Ganesh Sadawarte to review the same.   Thanks.
            Hide
            Permalink
            Ganesh.sadawarte Ganesh Sadawarte (Inactive) added a comment -

            Hi Mohd Belal,

            I have done with code review and merge changes to develop branches

            CC-Swapnil Pandhare

            Thanks,
            Ganesh

            Show
            Ganesh.sadawarte Ganesh Sadawarte (Inactive) added a comment - Hi Mohd Belal , I have done with code review and merge changes to develop branches CC- Swapnil Pandhare Thanks, Ganesh
            Hide
            Permalink
            mohd.belal Mohd Belal (Inactive) added a comment -

            Thanks, Ganesh Sadawarte for merging into develop.

            Marking this into Code Review.

             

             

            Show
            mohd.belal Mohd Belal (Inactive) added a comment - Thanks, Ganesh Sadawarte for merging into develop. Marking this into Code Review.    
            Hide
            Permalink
            jayshree.nagpure Jayshree Nagpure (Inactive) added a comment -

            Verified on preproduction.

            Verified below scenario :

            • For invalid email address system is showing validation message.
            • For valid email address notification setting are saved.

            Issue is fixed.
            PFA -

            Marking item state as Done.

            cc - Priya Dhamande

            Show
            jayshree.nagpure Jayshree Nagpure (Inactive) added a comment - Verified on preproduction. Verified below scenario : For invalid email address system is showing validation message. For valid email address notification setting are saved. Issue is fixed. PFA - Marking item state as Done. cc - Priya Dhamande

              People

              Assignee:
              mohd.belal Mohd Belal (Inactive)
              Reporter:
              jayshree.nagpure Jayshree Nagpure (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 4h
                  4h
                  Remaining:
                  Time Spent - 1h 20m Remaining Estimate - 2h 40m
                  2h 40m
                  Logged:
                  Time Spent - 1h 20m Remaining Estimate - 2h 40m
                  1h 20m