Skip to content

Bump LDK to 0.3.0-beta1 - #1008

Open
tnull wants to merge 1 commit into
lightningdevkit:mainfrom
tnull:2026-07-bump-ldk-to-0.3-beta
Open

Bump LDK to 0.3.0-beta1#1008
tnull wants to merge 1 commit into
lightningdevkit:mainfrom
tnull:2026-07-bump-ldk-to-0.3-beta

Conversation

@tnull

@tnull tnull commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Track the first LDK 0.3 beta and its matching payment-instructions revision. Preserve Esplora configuration across the client versions and expose splice negotiation only to local contributors, as defined by the beta API.

Co-Authored-By: HAL 9000

Track the first LDK 0.3 beta and its matching payment-instructions
revision. Preserve Esplora configuration across the client versions and
expose splice negotiation only to local contributors, as defined by the
beta API.

Co-Authored-By: HAL 9000
@tnull
tnull requested a review from jkczyz July 30, 2026 12:24
@ldk-reviews-bot

ldk-reviews-bot commented Jul 30, 2026

Copy link
Copy Markdown

👋 Thanks for assigning @jkczyz as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

Comment thread src/event.rs
/// A channel splice with local inputs or outputs has been negotiated and the funding
/// transaction is pending confirmation on-chain.
///
/// This event is not emitted when only the counterparty contributes to a splice.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same for SpliceNegotiationFailed?

Comment thread src/event.rs
skimmed_fee_msat,
claim_from_onchain_tx,
outbound_amount_forwarded_msat,
outbound_amount_forwarded_msat: Some(outbound_amount_forwarded_msat),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we remove the Option from the LDK Node event, too?

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.

3 participants