Skip to content
This repository has been archived by the owner on Dec 15, 2023. It is now read-only.

Fix estimate fee #88

Merged
merged 3 commits into from
Apr 21, 2022
Merged

Fix estimate fee #88

merged 3 commits into from
Apr 21, 2022

Conversation

badurinantun
Copy link
Contributor

@badurinantun badurinantun commented Apr 20, 2022 •
edited
Loading

Usage related changes

  • Fixed fee estimate not to depend on the fixed TRANSACTION_VERSION

Development related changes

  • None

Checklist:

  • Performed a self-review of the code
  • Rebased to the base branch
  • Documented the changes
  • Updated the tests

@badurinantun badurinantun requested a review from FabijanC April 20, 2022 16:34
@badurinantun badurinantun self-assigned this Apr 20, 2022
FabijanC
FabijanC suggested changes Apr 21, 2022
View reviewed changes
starknet_devnet/starknet_wrapper.py Outdated Show resolved Hide resolved
@badurinantun badurinantun requested a review from FabijanC April 21, 2022 08:47
@FabijanC FabijanC merged commit 12a929a into master Apr 21, 2022
@FabijanC FabijanC deleted the fix-estimate-fee branch April 21, 2022 09:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@FabijanC FabijanC FabijanC approved these changes

Assignees

@badurinantun badurinantun

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@badurinantun @FabijanC