[DMS-2058] Remove verification for user company mapping - Workterra Jira

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

Done

Harish.Sharma (Inactive)

Activity

Ascending order - Click to sort in descending order

Narendra Kumar created issue - 06/Jan/20 01:33 AM

Narendra Kumar made changes - 07/Jan/20 02:14 AM

Field	Original Value	New Value
Status	To Do [ 10000 ]	In Progress [ 3 ]

Hide

Permalink

Narendra Kumar added a comment - 07/Jan/20 02:37 AM

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 added a comment - 07/Jan/20 02:37 AM 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

Narendra Kumar made changes - 07/Jan/20 02:37 AM

Remaining Estimate	8h [ 28800 ]	4h [ 14400 ]
Time Spent		4h [ 14400 ]
Worklog Id	220174 [ 220174 ]

Permalink

Narendra Kumar logged work - 07/Jan/20 03:48 AM

Time Spent:

4h

Removed the validation for verification of user and company mapping.

Hide

Permalink

Narendra Kumar added a comment - 07/Jan/20 09:44 PM

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 added a comment - 07/Jan/20 09:44 PM 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

Narendra Kumar made changes - 07/Jan/20 09:44 PM

Status

In Progress [ 3 ]

Code Review [ 11801 ]

Narendra Kumar made changes - 07/Jan/20 09:45 PM

Remaining Estimate	4h [ 14400 ]	0h [ 0 ]
Time Spent	4h [ 14400 ]	11h [ 39600 ]
Worklog Id	220396 [ 220396 ]

Permalink

Narendra Kumar logged work - 08/Jan/20 03:44 AM

Time Spent:

7h

Done with the changes

Hide

Permalink

Narendra Kumar added a comment - 08/Jan/20 09:35 PM

Hi Harshveer Singh

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

Thanks

Narendra

Show

Narendra Kumar added a comment - 08/Jan/20 09:35 PM Hi Harshveer Singh As we have discussed, I have added the check for duplicate email. Thanks Narendra

Permalink

Narendra Kumar logged work - 08/Jan/20 09:36 PM

Time Spent:

2h

Added check for duplicate email.

Narendra Kumar made changes - 08/Jan/20 09:36 PM

Time Spent	11h [ 39600 ]	13h [ 46800 ]
Worklog Id	220595 [ 220595 ]

Narendra Kumar made changes - 08/Jan/20 09:37 PM

Remaining Estimate	0h [ 0 ]	3h [ 10800 ]
Original Estimate	8h [ 28800 ]	16h [ 57600 ]

Ashish Durani made changes - 08/Jan/20 10:57 PM

Status

Code Review [ 11801 ]

In QA Testing [ 11901 ]

Ashish Durani made changes - 08/Jan/20 10:58 PM

Assignee

Narendra Kumar [ narendra.kumar ]

Prateek [ prateeks ]

Prateek made changes - 10/Jan/20 03:08 AM

Status

In QA Testing [ 11901 ]

QA Reopen [ 11306 ]

Prateek made changes - 10/Jan/20 03:13 AM

Status

QA Reopen [ 11306 ]

Prateek made changes - 10/Jan/20 03:14 AM

Assignee

Prateek [ prateeks ]

Narendra Kumar [ narendra.kumar ]

Hide

Permalink

Prateek added a comment - 10/Jan/20 03:16 AM

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

Prateek added a comment - 10/Jan/20 03:16 AM 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

Narendra Kumar made changes - 21/Jan/20 02:34 AM

Status

QA Reopen [ 11306 ]

In Progress [ 3 ]

Hide

Permalink

Narendra Kumar added a comment - 21/Jan/20 05:19 AM

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 added a comment - 21/Jan/20 05:19 AM 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

Narendra Kumar made changes - 21/Jan/20 05:19 AM

Status

In Progress [ 3 ]

In QA Testing [ 11901 ]

Narendra Kumar made changes - 21/Jan/20 05:19 AM

Assignee

Narendra Kumar [ narendra.kumar ]

Prateek [ prateeks ]

Permalink

Prateek logged work - 22/Jan/20 02:44 AM

Time Spent:

0.75h

<No comment>

Hide

Permalink

Prateek added a comment - 22/Jan/20 02:44 AM

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

Prateek added a comment - 22/Jan/20 02:44 AM 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

Prateek made changes - 22/Jan/20 02:44 AM

Resolution		Done [ 10000 ]
Status	In QA Testing [ 11901 ]	Done [ 10001 ]

Prateek made changes - 22/Jan/20 02:44 AM

Remaining Estimate	3h [ 10800 ]	2.25h [ 8100 ]
Time Spent	13h [ 46800 ]	13.75h [ 49500 ]
Worklog Id	222860 [ 222860 ]

Transition	Time In Source Status	Execution Times

Narendra Kumar made transition - 07/Jan/20 02:14 AM

To Do

In Progress

1d 41m

Narendra Kumar made transition - 07/Jan/20 09:44 PM

In Progress

Code Review

19h 29m

Ashish Durani made transition - 08/Jan/20 10:57 PM

Code Review

In QA Testing

1d 1h 12m

Prateek made transition - 10/Jan/20 03:08 AM

In QA Testing

QA Reopen

1d 4h 11m

Prateek made transition - 10/Jan/20 03:13 AM

QA Reopen

5m 19s

Narendra Kumar made transition - 21/Jan/20 02:34 AM

QA Reopen

In Progress

10d 23h 21m

Narendra Kumar made transition - 21/Jan/20 05:19 AM

In Progress

In QA Testing

2h 45m

Prateek made transition - 22/Jan/20 02:44 AM

In QA Testing

Done

21h 24m

People

Assignee:: Prateek

Reporter:: Narendra Kumar

Votes:: 0 Vote for this issue

Watchers:: 1 Start watching this issue

Dates

Created:: 06/Jan/20 01:33 AM

Updated:: 22/Jan/20 02:44 AM

Resolved:: 22/Jan/20 02:44 AM

Time Tracking

Estimated:

24h

Remaining:

9.75h

Logged:

14.25h

Include sub-tasks

Details

Description

Attachments

Attachments

Sub-Tasks

Activity

People

Dates

Time Tracking