Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace sendRawTransaction() with boolean param for Bitcoin Core 0.19+ #63

Merged

Conversation

msgilligan
Copy link
Member

  • Deprecate sendRawTransaction(Transaction tx, Boolean allowHighFees)
  • Replace with sendRawTransaction(Transaction tx, Coin maxFeeRate)
    (available in Bitcoin Core 0.19 and later)
  • Create temporary checkForLegacyBitcoinCore() method in RegTestFundingSource
  • Remove deprecated generate() methods in BitcoinExtendedClient
  • Related and semi-related code cleanup in BitcoinClient, BitcoinExtendedClient,
    and BitcoinExtnededClientSpec

* Deprecate sendRawTransaction(Transaction, Boolean allowHighFees)
* Replace with sendRawTransaction(Transaction, Coin maxFeeRate)
  (available in Bitcoin Core 0.19 and later)
* Create temporary checkForLegacyBitcoinCore() method in RegTestFundingSource
* Remove deprecated generate() methods in BitcoinExtendedClient
* Related and semi-related code cleanup in BitcoinClient, BitcoinExtendedClient,
  and BitcoinExtnededClientSpec
@msgilligan msgilligan requested a review from dexX7 July 2, 2020 23:07
@msgilligan msgilligan merged commit 799a718 into master Jul 3, 2020
@msgilligan msgilligan deleted the msgilligan-deprecate-sendrawtransaction-boolean-arg branch July 3, 2020 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.

None yet

2 participants