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

Allow fee estimation for version 0 and 1 #268

Merged

FabijanC merged 2 commits into master from fix-estimate-fee-0.10.0

Sep 9, 2022

Collaborator

FabijanC commented Sep 9, 2022 •

edited

Loading

Usage related changes

Closes Allow fee estimation for version 0 and 1 #268
Resolve starknet.js fee estimation issue

Development related changes

Load request with Transaction, if that fails, default to InvokeFunction
Move 0.10.0 e2e script to scripts

Checklist:

Applied formatting - ./scripts/format.sh
No linter errors - ./scripts/lint.sh
Performed code self-review
Rebased to the last commit of the target branch (or merged it into my branch)
Documented the changes
Linked the issues which this PR resolves
Updated the tests
All tests are passing


          Allow fee estimation for version 0 and 1

6f31ebd

FabijanC marked this pull request as ready for review

September 9, 2022 15:27


          Fix invalid data test

3fe0b37

FabijanC merged commit a0d2214 into master

FabijanC deleted the fix-estimate-fee-0.10.0 branch

September 9, 2022 15:51

FabijanC linked an issue

that may be closed by this pull request

estimate_fee problem regarding 0.10 changes #266

Closed

FabijanC mentioned this pull request

estimate_fee problem regarding 0.10 changes #266

Closed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet