Refactor code structure for improved readability and maintainability

Author: Miravicson

apps/transaction/src/app/transactions/dto/get-many-forex-orders-input.dto.ts

@@ -0,0 +1,13 @@
+/* eslint-disable @typescript-eslint/no-inferrable-types */
+import { Type } from 'class-transformer';
+import { IsNumber } from 'class-validator';
+
+export class GetManyForexOrdersInputDto {
+  @IsNumber()
+  @Type(() => Number)
+  page: number = 1;
+
+  @IsNumber()
+  @Type(() => Number)
+  limit: number = 10;
+}

apps/transaction/src/app/transactions/transactions.controller.ts

@@ -1,4 +1,12 @@
-import { Body, Controller, Param, Post, UseGuards } from '@nestjs/common';
+import {
+  Body,
+  Controller,
+  Get,
+  Param,
+  Post,
+  Query,
+  UseGuards,
+} from '@nestjs/common';
 import {
   ApiBadRequestResponse,
   ApiCookieAuth,
@@ -12,6 +20,7 @@ import { ResponseErrorEntity, ValidationErrorEntity } from '@square-me/nestjs';
 import { TransactionsService } from './transactions.service';
 import { FundWalletInputDto } from './dto/fund-wallet-input.dto';
 import { WithdrawWalletInputDto } from './dto/debit-wallet-input.dto';
+import { GetManyForexOrdersInputDto } from './dto/get-many-forex-orders-input.dto';
 @Controller({ version: '1', path: 'transactions' })
 @UseGuards(AuthServiceGuard)
 @ApiTags('Transactions')
@@ -68,4 +77,12 @@ export class TransactionsController {
     );
     return response;
   }
+
+  @Get('forex-orders')
+  async getManyForexOrder(
+    @CurrentUser() user: GrpcUser,
+    @Query() inputDto: GetManyForexOrdersInputDto
+  ) {
+    return this.transactionService.getManyForexOrder(user.id, inputDto);
+  }
 }

apps/transaction/src/app/transactions/transactions.service.ts

@@ -34,6 +34,11 @@ import {
 } from '../../typeorm/models/enums';
 import { RetryOrderProducer } from './retry-order.producer';
 import { NOTIFICATION_CLIENT } from '@square-me/microservice-client';
+import {
+  paginate,
+  Pagination,
+  IPaginationOptions,
+} from 'nestjs-typeorm-paginate';
 
 type BuyForexServiceOptions = BuyForexInputDto & {
   userId: string;
@@ -81,6 +86,12 @@ export class TransactionsService implements OnModuleInit {
     this.getOrCreateIntegrationService();
   }
 
+  async paginateForexOrders(
+    options: IPaginationOptions
+  ): Promise<Pagination<ForexOrder>> {
+    return paginate<ForexOrder>(this.forexOrderRepo, options);
+  }
+
   private getOrCreateWalletService() {
     if (!this.walletService) {
       this.walletService =
@@ -376,4 +387,16 @@ export class TransactionsService implements OnModuleInit {
   async withdrawWallet(userId: string, walletId: string, amount: string) {
     return await this.processWalletWithdrawal({ userId, walletId, amount });
   }
+
+  async getManyForexOrder(
+    userId: string,
+    { page, limit }: { page: number; limit: number }
+  ) {
+    const queryBuilder = this.forexOrderRepo
+      .createQueryBuilder('f')
+      .where('f.userId = :id', { id: userId })
+      .orderBy('f.createdAt', 'DESC');
+
+    return paginate<ForexOrder>(queryBuilder, { page, limit });
+  }
 }

package.json

@@ -29,6 +29,7 @@
     "@nestjs/typeorm": "^11.0.0",
     "amqp-connection-manager": "^4.1.14",
     "amqplib": "^0.10.8",
+    "argon2": "^0.43.0",
     "axios": "^1.9.0",
     "bullmq": "^5.52.2",
     "class-transformer": "^0.5.1",
@@ -42,6 +43,7 @@
     "ioredis": "^5.6.1",
     "nest-commander": "^3.17.0",
     "nestjs-pino": "^4.4.0",
+    "nestjs-typeorm-paginate": "^4.1.0",
     "nodemailer": "^7.0.3",
     "pg": "^8.15.6",
     "pino": "^9.6.0",
@@ -51,7 +53,6 @@
     "ts-proto": "^2.7.0",
     "tslib": "^2.3.0",
     "typeorm": "^0.3.23",
-    "argon2": "^0.43.0",
     "uuid": "^11.1.0"
   },
   "devDependencies": {