Skip to content

Conversation

@dcousens
Copy link
Contributor

Simple enough fix.

Added a test fixture to verify the sequence numbers, transaction version and locktime.

Fixes #341.

Happy to merge with 1 ACK.

@dcousens dcousens changed the title Fix for #341 TransactionBuilder to copy version/locktime in fromTransaction Jan 23, 2015
@coveralls
Copy link

Coverage Status

Coverage increased (+0.0%) to 98.25% when pulling cbf8c6f on txbfix into 510212a on master.

weilu added a commit that referenced this pull request Jan 24, 2015
TransactionBuilder to copy version/locktime in fromTransaction
@weilu weilu merged commit 93714a2 into master Jan 24, 2015
@weilu weilu deleted the txbfix branch January 24, 2015 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TransactionBuilder.fromTransaction doesn't copy locktime

4 participants