Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increasing storage estimation cost after doing an estimation fails #2318

Closed
zamrokk opened this issue Jan 25, 2023 · 2 comments · Fixed by #2329
Closed

Increasing storage estimation cost after doing an estimation fails #2318

zamrokk opened this issue Jan 25, 2023 · 2 comments · Fixed by #2329
Assignees
Labels
bug Something isn't working
Projects
Milestone

Comments

@zamrokk
Copy link

zamrokk commented Jan 25, 2023

I took the example from https://tezostaquito.io/docs/transaction_limits/#setting-the-limits

my code here, I just want to increase the storage limit estimation x 2.5 :

const preparedCall = await mainWalletType!.methods.revealPlay(
        encryptedAction as bytes,
        new BigNumber(secretAction.secret) as nat,
        current_session!.current_round,
        session_id
      );

      const { gasLimit, storageLimit, suggestedFeeMutez } =
        await Tezos.estimate.transfer(preparedCall.toTransferParams());

      console.log({ gasLimit, storageLimit, suggestedFeeMutez });
      const op = await preparedCall.send({
        gasLimit,
        fee: suggestedFeeMutez,
        storageLimit: storageLimit * 2.5,
      });
      await op?.confirmation();

image

I seems that the fees have disappeared .. ?

@zamrokk zamrokk added the bug Something isn't working label Jan 25, 2023
@zamrokk
Copy link
Author

zamrokk commented Jan 25, 2023

proof

image

@Innkst Innkst added this to To do in dev via automation Jan 30, 2023
@Innkst Innkst added this to the v16 milestone Jan 30, 2023
@dsawali dsawali moved this from To do to In progress in dev Jan 31, 2023
@dsawali dsawali self-assigned this Jan 31, 2023
@dsawali dsawali mentioned this issue Jan 31, 2023
9 tasks
@dsawali dsawali moved this from In progress to Review in progress in dev Jan 31, 2023
@dsawali dsawali moved this from Review in progress to Done in dev Feb 1, 2023
@zamrokk
Copy link
Author

zamrokk commented Feb 1, 2023

image

@Innkst Innkst closed this as completed Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
dev
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants