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

v28: Add support for submitpackage #14

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

tcharding
Copy link
Member

@tcharding tcharding commented Nov 27, 2024

Bitcoin Core introduced a new RPC method submitpackage. Add support for it including integration testing.

This work was originally done by tnull a while ago over on the old rust-bitcoind-json-rpc repository (PR: rust-bitcoin/rust-bitcoind-json-rpc#23).

I took his changes and updated the code with the ideas I've learned recently fleshing out v17.

Patch includes <Co-developed-by> tag.

Close: #7

@tcharding
Copy link
Member Author

cc @tnull

///
/// # Returns
///
/// The receive address and the transaction.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In 3e17653:

Trailing whitespace.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Were are the git hooks!!!

@apoelstra
Copy link
Member

Otherwise 3e17653 looks good.

Also CI is failing. I didn't look at why.

Bitcoin Core introduced a new RPC method `submitpackage`. Add support
for it including integration testing.

This work was originally done by tnull a while ago over on the old
`rust-bitcoind-json-rpc` repository (PR: rust-bitcoin#23).

I took his changes and updated the code with the ideas I've learned
recently fleshing out v17.

<Co-developed-by>: Elias Rohrer <dev@tnull.de>
@tcharding
Copy link
Member Author

Thanks man, CI was docs. Should be fixed now.

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 7c3024c; successfully ran local tests

@apoelstra apoelstra merged commit dada062 into rust-bitcoin:master Nov 28, 2024
32 checks passed
@tcharding tcharding deleted the 11-28-submitpackage branch November 28, 2024 22:35
@tcharding
Copy link
Member Author

The PR description is wrong - submitpackage exists all the way back to v24.

@tnull
Copy link
Contributor

tnull commented Feb 13, 2025

The PR description is wrong - submitpackage exists all the way back to v24.

Sorry! 😁

@tcharding
Copy link
Member Author

No stress man, shit happens.

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.

Add support for submitpackage
3 participants