fix: all sql queries should be under one transaction

bigcapitalhq · Jun 6, 2024 · 3dadbea · 3dadbea
1 parent 494d2c1
commit 3dadbea
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/server/src/services/Banking/Plaid/PlaidSyncDB.ts b/packages/server/src/services/Banking/Plaid/PlaidSyncDB.ts
@@ -46,7 +46,7 @@ export class PlaidSyncDb {
  trx?: Knex.Transaction
  ) {
  const { Account } = this.tenancy.models(tenantId);
- const plaidAccount = Account.query().findOne(
+ const plaidAccount = await Account.query().findOne(
  'plaidAccountId',
  createBankAccountDTO.plaidAccountId
  );
@@ -101,11 +101,11 @@ export class PlaidSyncDb {
  ): Promise<void> {
  const { Account } = this.tenancy.models(tenantId);
 
- const cashflowAccount = await Account.query()
+ const cashflowAccount = await Account.query(trx)
  .findOne({ plaidAccountId })
  .throwIfNotFound();
 
- const openingEquityBalance = await Account.query().findOne(
+ const openingEquityBalance = await Account.query(trx).findOne(
  'slug',
  'opening-balance-equity'
  );