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

Remove verification for user company mapping

    Details

    • Type: New Feature
    • Status: Done
    • Priority: Medium
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: Winter 2020 Enhancements
    • Labels:
      None
    • Company:
      All Clients/Multiple Clients

      Description

      I'm still not able to send the document for the candidate from multiple companies from UI Side.

      Remove verification for user company mapping

        Attachments

          Activity

          Hide
          narendra.kumar Narendra Kumar added a comment -

          Hi Harshveer Singh

          I have removed the validation for verification of user and company mapping.

          But at the backend side mapping not working with multiple companies. Please check it.

           

           

          Thanks

          Narendra

          Show
          narendra.kumar Narendra Kumar added a comment - Hi Harshveer Singh I have removed the validation for verification of user and company mapping. But at the backend side mapping not working with multiple companies. Please check it.     Thanks Narendra
          Hide
          narendra.kumar Narendra Kumar added a comment -

          Hi Harshveer Singh

          I have done with the following:

          • Added UserLoginId in the person table.
          • Added userCompany mapping.

          I'm not putting the check for the existing email check because we have already added the max communication Id in the person table. And this reference should exist in communication detail and then in the email table otherwise it can raise a problem.

           Assigned it PR to you please check it.

           

          cc: Samir

          Show
          narendra.kumar Narendra Kumar added a comment - Hi Harshveer Singh I have done with the following: Added UserLoginId in the person table. Added userCompany mapping. I'm not putting the check for the existing email check because we have already added the max communication Id in the person table. And this reference should exist in communication detail and then in the email table otherwise it can raise a problem.  Assigned it PR to you please check it.   cc: Samir
          Hide
          narendra.kumar Narendra Kumar added a comment -

          Hi Harshveer Singh

          As we have discussed, I have added the check for duplicate email.

           

          Thanks

          Narendra

          Show
          narendra.kumar Narendra Kumar added a comment - Hi Harshveer Singh As we have discussed, I have added the check for duplicate email.   Thanks Narendra
          Hide
          PrateekS Prateek added a comment -

          Hi Narendra Kumar

          We had logged in using SA and send mail to prateek.saxena@careerbuilder.com on careerbuilder test company.

          When we logged in with candidate (SAPR/SAPR), on company drop down, i am able to see only CB internal beta

          Show
          PrateekS Prateek added a comment - Hi Narendra Kumar We had logged in using SA and send mail to prateek.saxena@careerbuilder.com on careerbuilder test company. When we logged in with candidate (SAPR/SAPR), on company drop down, i am able to see only CB internal beta
          Hide
          narendra.kumar Narendra Kumar added a comment -

          Yes Prateek,

          But now you are not getting the error message that "Current user's company is not mapped to the given email address."

          Now you are able to send the document to the candidate from multiple companies. This was the only fixed in this ticket.

          cc: Samir

           

          Thanks

          Narendra

          Show
          narendra.kumar Narendra Kumar added a comment - Yes Prateek , But now you are not getting the error message that "Current user's company is not mapped to the given email address." Now you are able to send the document to the candidate from multiple companies. This was the only fixed in this ticket. cc: Samir   Thanks Narendra
          Hide
          PrateekS Prateek added a comment -

          Hi Narendra Kumar

          As per our discussion, we are closing this ticket as admin is able to send mail from multiple company to same user. and for drop down issue we will track it separately

          Show
          PrateekS Prateek added a comment - Hi Narendra Kumar As per our discussion, we are closing this ticket as admin is able to send mail from multiple company to same user. and for drop down issue we will track it separately

            People

            Assignee:
            PrateekS Prateek
            Reporter:
            narendra.kumar Narendra Kumar
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 24h
                24h
                Remaining:
                Time Spent - 14.25h Remaining Estimate - 9.75h
                9.75h
                Logged:
                Time Spent - 14.25h Remaining Estimate - 9.75h
                14.25h