Skip to content

Commit

Permalink
Merge pull request #902 from starknet-io/v6/fix/acc-version
Browse files Browse the repository at this point in the history
fix: estimateFeeBulk provided version
  • Loading branch information
tabaktoni committed Dec 19, 2023
2 parents f580edb + da717be commit 6fd010a
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/account/default.ts
Original file line number Diff line number Diff line change
Expand Up @@ -245,12 +245,15 @@ export class Account extends Provider implements AccountInterface {
invocations: Invocations,
details: UniversalDetails = {}
): Promise<EstimateFeeBulk> {
const { nonce, blockIdentifier } = details;
const { nonce, blockIdentifier, version } = details;
const accountInvocations = await this.accountInvocationsFactory(invocations, {
...v3Details(details),
versions: [
ETransactionVersion.F1, // non-sierra
this.getPreferredVersion(ETransactionVersion.F2, ETransactionVersion.F3), // sierra
toTransactionVersion(
this.getPreferredVersion(ETransactionVersion.F2, ETransactionVersion.F3),
version
), // sierra
],
nonce,
blockIdentifier,
Expand Down

0 comments on commit 6fd010a

Please sign in to comment.