Closed yknl closed 4 years ago
Merging #45 into master will decrease coverage by
1.74%
. The diff coverage is62.88%
.
@@ Coverage Diff @@
## master #45 +/- ##
==========================================
- Coverage 87.20% 85.45% -1.75%
==========================================
Files 25 26 +1
Lines 1321 1403 +82
Branches 238 250 +12
==========================================
+ Hits 1152 1199 +47
- Misses 168 203 +35
Partials 1 1
Impacted Files | Coverage Δ | |
---|---|---|
src/builders.ts | 75.16% <51.35%> (-22.46%) |
:arrow_down: |
src/authorization.ts | 76.70% <100.00%> (+0.26%) |
:arrow_up: |
src/index.ts | 100.00% <100.00%> (ø) |
|
src/network.ts | 100.00% <100.00%> (ø) |
|
src/transaction.ts | 78.57% <100.00%> (-0.55%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update cfac1ba...b6b57fe. Read the comment docs.
This adds the
estimateTransfer
function to fetch and calculate the estimated fee for the token transfer transaction. ThemakeSTXTokenTransfer
and smart contract tx builders will now also estimate and set fees automatically if not specified. This necessitated changing the builder functions to async and moving the fee argument into the options object.Usage
To set fees manually
The estimate function can also be used independently
For details refer to issue #14
Type of Change
Does this introduce a breaking change?
Yes, the transaction builder functions are now all async. The function signatures have been changed as well, removing
fee
as an argument.Are documentation updates required?
Yes, readme updated.
Checklist
npm run test
passes