Skip to content

Commit f2412c8

Browse files
committed
Updated sendOTP logic
1 parent 6590e48 commit f2412c8

File tree

2 files changed

+15
-13
lines changed

2 files changed

+15
-13
lines changed

src/api/api.controller.ts

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,15 @@ export class ApiController {
8080
@Headers('x-application-id') applicationId?,
8181
): Promise<any> {
8282
let startTime = Date.now();
83+
84+
let status: any, isWhatsApp = false, countryCode, number;
85+
86+
if (params.phone.includes('-')) {
87+
[countryCode, number] = params.phone.split('-');
88+
params.phone = number;
89+
} else {
90+
number = params.phone;
91+
}
8392
if (applicationId) {
8493
const { total }: { total: number; users: Array<User> } =
8594
await this.fusionAuthService.getUsersByString(
@@ -102,16 +111,10 @@ export class ApiController {
102111
}
103112
}
104113

105-
let status: any, isWhatsApp = false, countryCode, number;
106114
// Check if phone number contains country code (e.g. 91-1234567890)
107115
if (params.deliveryType=='WA') {
108116
isWhatsApp = true;
109-
if (params.phone.includes('-')) {
110-
[countryCode, number] = params.phone.split('-');
111-
} else {
112-
number = params.phone;
113-
}
114-
params.phone = number;
117+
115118
status = await this.gupshupWhatsappService.sendWhatsappOTP({
116119
phone: number,
117120
template: null,

src/api/api.service.ts

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -565,6 +565,10 @@ export class ApiService {
565565
let otp = loginDto.password;
566566
let phone = loginDto.loginId;
567567
let countryCode, number;
568+
if (phone.includes('-')) {
569+
[countryCode, number] = phone.split('-');
570+
phone = number;
571+
}
568572
const salt = this.configResolverService.getSalt(loginDto.applicationId);
569573
let verifyOTPResult;
570574
if(
@@ -577,12 +581,7 @@ export class ApiService {
577581
else
578582
verifyOTPResult = {status: SMSResponseStatus.failure}
579583
} else if (loginDto.deliveryType=='WA') {
580-
if(phone.includes('-')){
581-
[countryCode, number] = phone.split('-');
582-
} else {
583-
number = phone
584-
}
585-
loginDto.loginId = number;
584+
loginDto.loginId = phone;
586585
const status: any = await this.gupshupWhatsappService.verifyWhatsappOTP(loginDto.loginId, loginDto.password);
587586
if(status.status == 'success') {
588587
verifyOTPResult = {status: SMSResponseStatus.success}

0 commit comments

Comments
 (0)