fix: adjust suggested max fee response type

starknet-io · Mar 30, 2023 · 78b1f81 · 78b1f81
1 parent 2d7f16e
commit 78b1f81
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/account/default.ts b/src/account/default.ts
@@ -25,6 +25,7 @@ import {
   InvocationsSignerDetails,
   InvokeFunctionResponse,
   MultiDeployContractResponse,
+  Nonce,
   Signature,
   TransactionBulk,
   TransactionStatus,
@@ -66,7 +67,7 @@ export class Account extends Provider implements AccountInterface {
         : pkOrSigner;
   }
 
-  public async getNonce(blockIdentifier?: BlockIdentifier): Promise<BigNumberish> {
+  public async getNonce(blockIdentifier?: BlockIdentifier): Promise<Nonce> {
     return super.getNonceForAddress(this.address, blockIdentifier);
   }
 
@@ -512,7 +513,7 @@ export class Account extends Provider implements AccountInterface {
         break;
     }
 
-    return feeEstimate.suggestedMaxFee.toString();
+    return feeEstimate.suggestedMaxFee;
   }
 
   /**

diff --git a/src/account/interface.ts b/src/account/interface.ts
@@ -19,6 +19,7 @@ import {
   InvocationsDetails,
   InvokeFunctionResponse,
   MultiDeployContractResponse,
+  Nonce,
   Signature,
   TransactionBulk,
   TransactionSimulation,
@@ -299,7 +300,7 @@ export abstract class AccountInterface extends ProviderInterface {
    * @param  {BlockIdentifier} blockIdentifier - optional blockIdentifier. Defaults to 'pending'
    * @returns nonce of the account
    */
-  public abstract getNonce(blockIdentifier?: BlockIdentifier): Promise<BigNumberish>;
+  public abstract getNonce(blockIdentifier?: BlockIdentifier): Promise<Nonce>;
 
   /**
    * Gets Suggested Max Fee based on the transaction type
@@ -311,7 +312,7 @@ export abstract class AccountInterface extends ProviderInterface {
   public abstract getSuggestedMaxFee(
     estimateFeeAction: EstimateFeeAction,
     details: EstimateFeeDetails
-  ): Promise<BigNumberish>;
+  ): Promise<bigint>;
 
   /**
    * Simulates the transaction and returns the transaction trace and estimated fee.