Employee Profile Sharing Dropdown List Delay

[telusdcinco]

It's been noticed that the dropdown list for sharing employees in My Team's Employee Profile Sharing modal page, is taking too long to return values. This part of the application have remained almost the same for the past 2 years, and can be updated to improve the performance by applying the changes below.

• not using LIKE in the SQL when there is no search keyword entered
• improve the SQL joins or use a newer/recent static table for the return employee listing
• add index to the SQL for consistent searching patterns

[jessicahjwu]

Related to #1304

[telusdcinco]

MIGRATION NOTES:

Code change only.

[telusdcinco]

@jessicahjwu @steffipravasi

The fix works very well and the dropdown is very responsive.

Should we change the behavior of the dropdown to not display anything when the user have not entered a single letter, so that the dropdown do not display same default full list as shown below?

[telusdcinco]

Fixed the format issue for the missing email address.

Also updated the name to use the Employee Demo name and not the User Profile name.

[steffipravasi]

@telusdcinco 1) The shared with list again won't load in my current goals

[telusdcinco]

@steffipravasi This ticket is for the My Teams page issue. I think the My Goal is a different ticket.

[steffipravasi]

The list in my goal is also loading now

[telusdcinco]

@jessicahjwu @steffipravasi Updated the dropdown search to allow searching from both employee demo or user profile names. User can now search using "John Smith" or "Smith,John" formats.

[telusdcinco]

@jessicahjwu There is significant performance degradation on the dropdown by searching in 2 name formats. Recommend to simplify the search by first+last name, and displaying the result in first+last name format also. This should make the resulting name format consistent, and users will see the the same name format as they type.

[telusdcinco]

@jessicahjwu MyTeam Share Profile dropdown reverted to yesterday's version which passed Test and pending Migration to Training. Please do a quick validation in Test, before I move to Training.

[telusdcinco]

Deployed to Training.

[telusdcinco]

Found code difference implemented in #1155, not found in Training and/or Prod. Pushed the missing code to Training.