Closed yknl closed 4 years ago
Merging #99 into master will decrease coverage by
1.39%
. The diff coverage is69.00%
.
@@ Coverage Diff @@
## master #99 +/- ##
==========================================
- Coverage 84.10% 82.70% -1.40%
==========================================
Files 28 28
Lines 1812 2024 +212
Branches 367 415 +48
==========================================
+ Hits 1524 1674 +150
- Misses 286 346 +60
- Partials 2 4 +2
Impacted Files | Coverage Δ | |
---|---|---|
src/errors.ts | 14.28% <14.28%> (ø) |
|
src/signer.ts | 73.21% <61.53%> (-4.06%) |
:arrow_down: |
src/keys.ts | 90.52% <70.00%> (-5.48%) |
:arrow_down: |
src/authorization.ts | 77.82% <74.11%> (+0.72%) |
:arrow_up: |
src/transaction.ts | 78.09% <77.27%> (-1.22%) |
:arrow_down: |
src/builders.ts | 77.06% <83.01%> (+0.92%) |
:arrow_up: |
src/types.ts | 83.18% <100.00%> (+0.15%) |
:arrow_up: |
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 9981f63...fa817f4. Read the comment docs.
@yknl could we get this merged and released before the refactor into the blockstack.js monorepo? Need it for use in https://github.com/blockstack/stacks-blockchain-api/pull/176
This PR will enable the construction of sponsored transactions where a second "sponsor" signer can pay the fees of a transaction signed by the origin.
To create a transaction that can be sponsored, set
sponsored = true
in the transaction builder options:Next, pass the serialized transaction to the sponsoring party and use the
sponsorTransaction
builder function to complete the transaction.https://github.com/blockstack/stacks-transactions-js/issues/7
Type of Change
Does this introduce a breaking change?
No
Are documentation updates required?
Yes
Checklist
npm run test
passes