Fix N+1 query issue in getRevenueBreakdown with pagination and aggreg…#943
Open
amankoli09 wants to merge 1 commit into
Open
Fix N+1 query issue in getRevenueBreakdown with pagination and aggreg…#943amankoli09 wants to merge 1 commit into
amankoli09 wants to merge 1 commit into
Annotations
10 errors and 4 warnings
|
|
|
Run lint:
src/payments/payouts/payouts.service.ts#L59
Replace `Refund,·'refund',·'refund.paymentId·=·payment.id·AND·refund.status·=·:refundStatus',·{·refundStatus:·RefundStatus.PROCESSED·}` with `⏎········Refund,⏎········'refund',⏎········'refund.paymentId·=·payment.id·AND·refund.status·=·:refundStatus',⏎········{·refundStatus:·RefundStatus.PROCESSED·},⏎······`
|
|
Run lint:
src/payments/payouts/payouts.service.ts#L58
Replace `Payment,·'payment',·'payment.courseId·=·course.id·AND·payment.status·=·:paymentStatus',·{·paymentStatus:·PaymentStatus.COMPLETED·}` with `⏎········Payment,⏎········'payment',⏎········'payment.courseId·=·course.id·AND·payment.status·=·:paymentStatus',⏎········{·paymentStatus:·PaymentStatus.COMPLETED·},⏎······`
|
|
Run lint:
src/payments/payouts/payouts.service.ts#L57
Insert `⏎······`
|
|
Run lint:
src/payments/payouts/payouts.service.ts#L44
Replace `⏎········'course.id·AS·"courseId"',⏎········'course.title·AS·"title"',⏎······` with `'course.id·AS·"courseId"',·'course.title·AS·"title"'`
|
|
Run lint:
src/payments/payouts/payouts.service.ts#L42
Replace `Refund,·'refund',·'refund.paymentId·=·payment.id·AND·refund.status·=·:refundStatus',·{·refundStatus:·RefundStatus.PROCESSED·}` with `⏎········Refund,⏎········'refund',⏎········'refund.paymentId·=·payment.id·AND·refund.status·=·:refundStatus',⏎········{·refundStatus:·RefundStatus.PROCESSED·},⏎······`
|
|
Run lint:
src/payments/payouts/payouts.service.ts#L41
Replace `Payment,·'payment',·'payment.courseId·=·course.id·AND·payment.status·=·:paymentStatus',·{·paymentStatus:·PaymentStatus.COMPLETED·}` with `⏎········Payment,⏎········'payment',⏎········'payment.courseId·=·course.id·AND·payment.status·=·:paymentStatus',⏎········{·paymentStatus:·PaymentStatus.COMPLETED·},⏎······`
|
|
Run lint:
src/payments/payouts/payouts.service.ts#L40
Insert `⏎······`
|
|
Run lint:
src/payments/payouts/payouts.service.spec.ts#L130
Replace `·courseId:·'course-2',·title:·'Course·Two',·grossRevenue:·'200.00',·refunds:·'0.00',·salesCount:·'1'` with `⏎··········courseId:·'course-2',⏎··········title:·'Course·Two',⏎··········grossRevenue:·'200.00',⏎··········refunds:·'0.00',⏎··········salesCount:·'1',⏎·······`
|
|
Run lint:
src/payments/payouts/payouts.service.spec.ts#L129
Replace `·courseId:·'course-1',·title:·'Course·One',·grossRevenue:·'250.00',·refunds:·'25.00',·salesCount:·'2'` with `⏎··········courseId:·'course-1',⏎··········title:·'Course·One',⏎··········grossRevenue:·'250.00',⏎··········refunds:·'25.00',⏎··········salesCount:·'2',⏎·······`
|
|
Complete job
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-node@v4, pnpm/action-setup@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Run lint:
src/workers/orchestration/worker-orchestration.service.ts#L1
'Inject' is defined but never used. Allowed unused vars must match /^_/u
|
|
Run lint:
src/workers/base/base.worker.ts#L1
'Inject' is defined but never used. Allowed unused vars must match /^_/u
|
|
Run lint:
src/payments/payouts/payouts.service.ts#L3
'In' is defined but never used. Allowed unused vars must match /^_/u
|
background
wait
wait-all
cancel
parallel
Loading