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

Deployment fee is not included in totalFees #20

Open
behrang opened this issue Feb 22, 2023 · 2 comments
Open

Deployment fee is not included in totalFees #20

behrang opened this issue Feb 22, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@behrang
Copy link

behrang commented Feb 22, 2023

When I try to deploy a child contract from a parent, the deployment fee is not returned in totalFees of the transaction. It looks like it only considers gas fees for processing the message, and not the gas fee required for providing stateInit and its deployment. Ideally, totalFees should include deployment fees, and maybe a separate field can be used for gas fees.

In my tests, I try to calculate the balance of parent smart contract, after deploying a new child, but since deployment cost is not considered in totalFees, I have to compromise and use an approximation.

@behrang behrang added the enhancement New feature or request label Feb 22, 2023
@behrang behrang changed the title Deployment fees is not included in totalFees Deployment fee is not included in totalFees Feb 22, 2023
@krigga
Copy link
Collaborator

krigga commented Feb 27, 2023

I've heard somewhere that totalFees are just wrong, but I'll have a look, cannot confirm or deny that.

@9oelM
Copy link

9oelM commented Oct 3, 2024

maybe related to #55

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants