Skip to content

Commit

Permalink
PR review
Browse files Browse the repository at this point in the history
  • Loading branch information
PatrickDinh committed Dec 15, 2024
1 parent 58f057f commit 37f0834
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
5 changes: 2 additions & 3 deletions src/transform.ts
Original file line number Diff line number Diff line change
Expand Up @@ -470,8 +470,7 @@ export function getIndexerTransactionFromAlgodTransaction(
paymentTransaction: new algosdk.indexerModels.TransactionPayment({
amount: BigInt(transaction.payment!.amount ?? 0), // The amount can be undefined
receiver: transaction.payment!.receiver.toString(),
// TODO: PD - confirm closeAmount
closeAmount: closeAmount !== undefined ? BigInt(closeAmount) : undefined,
closeAmount: closeAmount,
closeRemainderTo: transaction.payment!.closeRemainderTo?.toString(),
}),
}
Expand Down Expand Up @@ -553,7 +552,7 @@ export function getIndexerTransactionFromAlgodTransaction(
note: transaction.note,
lease: transaction.lease,
rekeyTo: transaction.rekeyTo,
closingAmount: closeAmount !== undefined ? BigInt(closeAmount) : undefined,
closingAmount: closeAmount,
createdApplicationIndex: createdAppId !== undefined ? BigInt(createdAppId) : undefined,
authAddr: blockTransaction.sgnr ? new algosdk.Address(blockTransaction.sgnr) : undefined,
innerTxns: blockTransaction.dt?.itx?.map((ibt) =>
Expand Down
2 changes: 1 addition & 1 deletion src/types/subscription.ts
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ export interface BalanceChange {
/** The address that the balance change is for. */
address: string
/** The asset ID of the balance change, or 0 for Algos. */
assetId: number | bigint
assetId: bigint
/** The amount of the balance change in smallest divisible unit or microAlgos. */
amount: bigint
/** The roles the account was playing that led to the balance change */
Expand Down

0 comments on commit 37f0834

Please sign in to comment.