Skip to content

Commit 5b016dc

Browse files
Merge pull request #734 from mahajanmahesh935/PaymentSuccessEmail
TASK #0000 : Remove logs from payment success flow
2 parents ecd8304 + a3824ed commit 5b016dc

2 files changed

Lines changed: 1 addition & 12 deletions

File tree

src/payments/services/payment.service.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -477,7 +477,7 @@ export class PaymentService {
477477
this.logger.log(
478478
`Certificate(s) generated and targets unlocked for user ${userId} (intent ${paymentIntentId}, courses: ${uniqueCourseIds.join(', ')})`,
479479
);
480-
this.logger.log(uniqueCourseIds.length,"uniqueCourseIds.length")
480+
481481
if (uniqueCourseIds.length > 0) {
482482
void this.premiumCertificatePurchaseEmailService
483483
.sendPremiumCertificatePurchasedIfEnabled({

src/payments/services/premium-certificate-purchase-email.service.ts

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,6 @@ export class PremiumCertificatePurchaseEmailService {
7272
certificateApiResponse: unknown;
7373
}): Promise<void> {
7474
if (!isSendEmailAfterPaymentSuccessEnabled(this.configService)) {
75-
this.logger.log(
76-
`Skipping premium certificate purchase email for intent ${params.paymentIntentId} because SEND_EMAIL_AFTER_PAYMENT_SUCCESS is not enabled`,
77-
);
7875
return;
7976
}
8077

@@ -139,10 +136,6 @@ export class PremiumCertificatePurchaseEmailService {
139136
},
140137
};
141138

142-
this.logger.log(
143-
`Sending premium certificate purchase email (OnPremiumCertificatePurchased) for intent ${params.paymentIntentId}`,
144-
);
145-
146139
const mailSend = await this.notificationRequest.sendNotification(
147140
notificationPayload,
148141
);
@@ -154,10 +147,6 @@ export class PremiumCertificatePurchaseEmailService {
154147
);
155148
return;
156149
}
157-
158-
this.logger.log(
159-
`Premium certificate purchase email sent for intent ${params.paymentIntentId}`,
160-
);
161150
} catch (err) {
162151
const message = err instanceof Error ? err.message : String(err);
163152
this.logger.error(

0 commit comments

Comments
 (0)