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

Expose bump transaction and channel close events #42

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

jjyr
Copy link
Collaborator

@jjyr jjyr commented Nov 25, 2024

  1. Repropose peer event broadcast Broadcast peer event #18 , now the channel broadcasts two new events: BumpChannelCloseTransaction and ChannelClosed
  2. The Js caller receives new events without any change.
    1. BumpChannelCloseTransaction event hints wallet to broadcast channel close transaction through backend mempool in case the node failed to broadcast tx through P2P network.
    2. ChannelClosed event hint wallet a potential force closed event is occured.

@jjyr jjyr requested review from Flouse, yuche and EthanYuan November 25, 2024 16:56
@Flouse Flouse requested a review from ShookLyngs November 26, 2024 01:05
@jjyr jjyr merged commit cb491dc into develop Nov 26, 2024
9 checks passed
@jjyr jjyr deleted the add-new-common-ln-events branch November 26, 2024 15:11
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