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

Enable opening Taproot Channels for LND #1296

Closed
saubyk opened this issue Oct 1, 2023 · 1 comment · Fixed by #1305
Closed

Enable opening Taproot Channels for LND #1296

saubyk opened this issue Oct 1, 2023 · 1 comment · Fixed by #1305
Labels
enhancement New feature or request LND

Comments

@saubyk
Copy link
Collaborator

saubyk commented Oct 1, 2023

LND v0.17 onwards will have the option to open Taproot channels, which should also be enabled via the RTL channel opening interface, provided the user has LND configured (protocol.simple-taproot-chans=true) to open taproot channels.

Openchannel API: https://lightning.engineering/api-docs/api/lnd/lightning/open-channel-sync
The argument to make it happen via the openchannel API is: commitment_type=5

The flag should be enabled by a toggle on the open channel modal under the Advanced Options section.

Currently, the additional argument to open taproot channels is also to set the private flag as true, but that should be left for the user to set when they are opening taproot channels. If the user doesn't set the flag, it will return an error which should indicate to the user that channels needs to be "private".

EDIT: updated the API argument for taproot channel

@saubyk saubyk added enhancement New feature or request LND labels Oct 1, 2023
@ShahanaFarooqui ShahanaFarooqui added this to the Release 0.14.1-beta milestone Oct 4, 2023
@ShahanaFarooqui ShahanaFarooqui linked a pull request Oct 5, 2023 that will close this issue
@ShahanaFarooqui
Copy link
Collaborator

Closing with PR #1305

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request LND
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants