laxmanhalaki/Re_Backend
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

title mapping to designation removed to avoid db errors

Browse Source
This commit is contained in:
laxmanhalaki 2026-04-20 17:14:25 +05:30
parent e7f52021c7
commit 2786d58bb6
3 changed files with 27 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/models/User.ts
View File

@ -173,11 +173,11 @@ User.init(
comment: 'Full Name for display' comment: 'Full Name for display'
}, },
department: { department: {
type: DataTypes.STRING(255), type: DataTypes.STRING(100),
allowNull: true allowNull: true
}, },
designation: { designation: {
type: DataTypes.STRING(255), type: DataTypes.STRING(100),
allowNull: true allowNull: true
}, },
phone: { phone: {

20
src/services/auth.service.ts
View File

@ -119,7 +119,7 @@ export class AuthService {
lastName: profile.lastName || undefined, lastName: profile.lastName || undefined,
displayName: profile.displayName || undefined, displayName: profile.displayName || undefined,
department: profile.department || undefined, department: profile.department || undefined,
designation: profile.title || profile.designation || undefined, designation: profile.designation || undefined,
phone: profile.mobilePhone || profile.phone || profile.phoneNumber || undefined, phone: profile.mobilePhone || profile.phone || profile.phoneNumber || undefined,
manager: profile.manager || undefined, // Store manager name if available manager: profile.manager || undefined, // Store manager name if available
jobTitle: profile.title || undefined, jobTitle: profile.title || undefined,
@ -209,8 +209,10 @@ export class AuthService {
if (oktaUser.department) { if (oktaUser.department) {
userData.department = oktaUser.department; userData.department = oktaUser.department;
} }
if (oktaUser.designation) {
userData.designation = oktaUser.designation;
}
if (oktaUser.title || oktaUser.designation) { if (oktaUser.title || oktaUser.designation) {
userData.designation = oktaUser.title || oktaUser.designation;
userData.jobTitle = oktaUser.title || oktaUser.designation; userData.jobTitle = oktaUser.title || oktaUser.designation;
} }
if (oktaUser.phone_number || oktaUser.phone) { if (oktaUser.phone_number || oktaUser.phone) {
@ -284,13 +286,13 @@ export class AuthService {
if (lastName) userUpdateData.lastName = lastName; if (lastName) userUpdateData.lastName = lastName;
if (displayName) userUpdateData.displayName = displayName; if (displayName) userUpdateData.displayName = displayName;
if (userData.employeeId) userUpdateData.employeeId = userData.employeeId; // Optional if (userData.employeeId) userUpdateData.employeeId = userData.employeeId; // Optional
if (userData.department) userUpdateData.department = userData.department; if (userData.department) userUpdateData.department = userData.department.substring(0, 100);
if (userData.designation) userUpdateData.designation = userData.designation; if (userData.designation) userUpdateData.designation = userData.designation.substring(0, 100);
if (userData.phone) userUpdateData.phone = userData.phone; if (userData.phone) userUpdateData.phone = userData.phone.substring(0, 20);
if (userData.manager) userUpdateData.manager = userData.manager; // Manager name from SSO if (userData.manager) userUpdateData.manager = userData.manager.substring(0, 200); // Manager name from SSO
if (userData.jobTitle) userUpdateData.jobTitle = userData.jobTitle; // Job title from SSO if (userData.jobTitle) userUpdateData.jobTitle = userData.jobTitle; // Job title from SSO (TEXT, no limit)
if (userData.postalAddress) userUpdateData.postalAddress = userData.postalAddress; // Address from SSO if (userData.postalAddress) userUpdateData.postalAddress = userData.postalAddress; // Address from SSO (TEXT, no limit)
if (userData.mobilePhone) userUpdateData.mobilePhone = userData.mobilePhone; // Mobile phone from SSO if (userData.mobilePhone) userUpdateData.mobilePhone = userData.mobilePhone.substring(0, 20); // Mobile phone from SSO
if (userData.employeeNumber || userData.dealerCode) { if (userData.employeeNumber || userData.dealerCode) {
userUpdateData.employeeNumber = userData.employeeNumber || userData.dealerCode; userUpdateData.employeeNumber = userData.employeeNumber || userData.dealerCode;
} }

27
src/services/user.service.ts
View File

@ -37,7 +37,7 @@ function extractOktaUserData(oktaUserResponse: any): SSOUserData | null {
lastName: profile.lastName || undefined, lastName: profile.lastName || undefined,
displayName: profile.displayName || undefined, displayName: profile.displayName || undefined,
department: profile.department || undefined, department: profile.department || undefined,
designation: profile.title || profile.designation || undefined, designation: profile.designation || undefined,
phone: profile.mobilePhone || profile.phone || profile.phoneNumber || undefined, phone: profile.mobilePhone || profile.phone || profile.phoneNumber || undefined,
manager: profile.manager || undefined, // Manager name from Okta manager: profile.manager || undefined, // Manager name from Okta
jobTitle: profile.title || undefined, jobTitle: profile.title || undefined,
@ -68,17 +68,17 @@ export class UserService {
const payload: any = { const payload: any = {
oktaSub: ssoData.oktaSub, oktaSub: ssoData.oktaSub,
employeeId: ssoData.employeeId || null, employeeId: ssoData.employeeId || null,
firstName: ssoData.firstName || null, firstName: ssoData.firstName ? ssoData.firstName.substring(0, 100) : null,
lastName: ssoData.lastName || null, lastName: ssoData.lastName ? ssoData.lastName.substring(0, 100) : null,
displayName: ssoData.displayName || null, displayName: ssoData.displayName ? ssoData.displayName.substring(0, 200) : null,
department: ssoData.department || null, department: ssoData.department ? ssoData.department.substring(0, 100) : null,
designation: ssoData.designation || null, designation: ssoData.designation ? ssoData.designation.substring(0, 100) : null,
phone: ssoData.phone || null, phone: ssoData.phone ? ssoData.phone.substring(0, 20) : null,
manager: ssoData.manager || null, manager: ssoData.manager ? ssoData.manager.substring(0, 200) : null,
jobTitle: ssoData.designation || ssoData.jobTitle || null, jobTitle: ssoData.jobTitle || ssoData.designation || null, // jobTitle is TEXT
postalAddress: ssoData.postalAddress || null, postalAddress: ssoData.postalAddress || null, // postalAddress is TEXT
mobilePhone: ssoData.mobilePhone || null, mobilePhone: ssoData.mobilePhone ? ssoData.mobilePhone.substring(0, 20) : null,
secondEmail: ssoData.secondEmail || null, secondEmail: ssoData.secondEmail ? ssoData.secondEmail.substring(0, 255) : null,
adGroups: ssoData.adGroups || null, adGroups: ssoData.adGroups || null,
lastLogin: now, lastLogin: now,
updatedAt: now, updatedAt: now,
@ -482,7 +482,8 @@ export class UserService {
lastName: oktaUserData.lastName || null, lastName: oktaUserData.lastName || null,
displayName: oktaUserData.displayName || email.split('@')[0], displayName: oktaUserData.displayName || email.split('@')[0],
department: oktaUserData.department || null, department: oktaUserData.department || null,
designation: oktaUserData.designation || oktaUserData.jobTitle || null, designation: oktaUserData.designation || null,
jobTitle: oktaUserData.jobTitle || oktaUserData.designation || null,
phone: oktaUserData.phone || oktaUserData.mobilePhone || null, phone: oktaUserData.phone || oktaUserData.mobilePhone || null,
isActive: true, isActive: true,
role: 'USER', role: 'USER',