Skip to content

Splice: Add splicein command#8856

Open
ddustin wants to merge 3 commits intoElementsProject:masterfrom
ddustin:ddustin/splicein
Open

Splice: Add splicein command#8856
ddustin wants to merge 3 commits intoElementsProject:masterfrom
ddustin:ddustin/splicein

Conversation

@ddustin
Copy link
Collaborator

@ddustin ddustin commented Jan 20, 2026

NOTE: This PR depends on the wallet & feerate fixes found in #8450 being merged.

The splicein is command is a convenience RPC method for splicing in funds from the onchain wallet, leveraging the splice script system to accomplish it.

@ddustin ddustin force-pushed the ddustin/splicein branch 3 times, most recently from 50c5603 to b75c33c Compare January 29, 2026 21:54
@ddustin ddustin added this to the v26.04 milestone Jan 29, 2026
@ddustin ddustin marked this pull request as ready for review February 2, 2026 03:40
@ddustin ddustin requested a review from cdecker as a code owner February 2, 2026 03:40
@madelinevibes madelinevibes added the Status::Ready for Review The work has been completed and is now awaiting evaluation or approval. label Feb 3, 2026
Copy link
Collaborator

@niftynei niftynei left a comment

Choose a reason for hiding this comment

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

ACK b75c33c

NULL))
return command_param_failed();

script = tal_fmt(NULL,
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is it! what we've been waiting for!

@sangbida sangbida added PLEASE clear CI 🫠 and removed Status::Ready for Review The work has been completed and is now awaiting evaluation or approval. labels Mar 18, 2026
ddustin added 3 commits March 20, 2026 17:56
Tests the splicein command
splicein is the convience method for splicing in using the splice script system.

Changelog-Added: New command `splicein` allows for convenient splicing funds into a channel
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants