Skip to content

Allow to override fee rates for onchain payments#434

Merged
tnull merged 1 commit into
lightningdevkit:mainfrom
tnull:2025-01-allow-to-override-feerate
Jan 27, 2025
Merged

Allow to override fee rates for onchain payments#434
tnull merged 1 commit into
lightningdevkit:mainfrom
tnull:2025-01-allow-to-override-feerate

Conversation

@tnull
Copy link
Copy Markdown
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
Comment thread src/payment/onchain.rs Outdated
Comment thread src/payment/onchain.rs Outdated
Comment thread src/payment/onchain.rs
Comment thread src/payment/onchain.rs Outdated
Comment thread src/payment/onchain.rs Outdated
Comment thread src/payment/onchain.rs Outdated
Comment thread src/uniffi_types.rs Outdated
Comment thread src/wallet/mod.rs Outdated
@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
Copy Markdown
Collaborator Author

tnull commented Jan 25, 2025

Squashed fixups without further changes.

@tnull tnull requested a review from jkczyz January 25, 2025 09:49
Comment thread src/payment/onchain.rs
@tnull tnull merged commit 80b7c92 into lightningdevkit:main Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants