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

Allow to override fee rates for onchain payments #434

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tnull
Copy link
Collaborator

@tnull tnull commented Jan 17, 2025

Fixes #176.

We allow to override our fee estimator in the send_to_address and send_all_to_address API methods. To this end, we implement a bindings-compatible wrapper around bitcoin::FeeRate.

@tnull tnull added this to the 0.5 milestone Jan 17, 2025
@tnull tnull requested a review from jkczyz January 17, 2025 12:14
@tnull tnull force-pushed the 2025-01-allow-to-override-feerate branch from c4afd11 to ec6bf80 Compare January 17, 2025 12:24
src/payment/onchain.rs Outdated Show resolved Hide resolved
src/payment/onchain.rs Outdated Show resolved Hide resolved
src/payment/onchain.rs Show resolved Hide resolved
src/payment/onchain.rs Outdated Show resolved Hide resolved
src/payment/onchain.rs Outdated Show resolved Hide resolved
src/payment/onchain.rs Outdated Show resolved Hide resolved
src/uniffi_types.rs Outdated Show resolved Hide resolved
src/wallet/mod.rs Outdated Show resolved Hide resolved
@tnull tnull force-pushed the 2025-01-allow-to-override-feerate branch 3 times, most recently from 1b0621d to bfbc72d Compare January 23, 2025 10:28
We allow to override our fee estimator in the `send_to_address` and
`send_all_to_address` API methods. To this end, we implement a
bindings-compatible wrapper around `bitcoin::FeeRate`.
@tnull tnull force-pushed the 2025-01-allow-to-override-feerate branch from bfbc72d to d6672b6 Compare January 25, 2025 09:47
@tnull
Copy link
Collaborator Author

tnull commented Jan 25, 2025

Squashed fixups without further changes.

@tnull tnull requested a review from jkczyz January 25, 2025 09:49
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.

Allow to override fee rate for on-chain operations
2 participants