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

Collaborative settlement of the DLC #50

Closed
1 of 2 tasks
Tracked by #215
luckysori opened this issue Jan 27, 2023 · 0 comments · Fixed by #222
Closed
1 of 2 tasks
Tracked by #215

Collaborative settlement of the DLC #50

luckysori opened this issue Jan 27, 2023 · 0 comments · Fixed by #222
Assignees

Comments

@luckysori
Copy link
Contributor

luckysori commented Jan 27, 2023

Implementation contingent on #49.

Two properties to be verified:

  • Funds correctly awarded to each wallet.
  • Can still create another DLC using those funds without going on-chain.
@luckysori luckysori added the MVP label Jan 27, 2023
@luckysori luckysori self-assigned this Mar 5, 2023
@bonomat bonomat mentioned this issue Mar 7, 2023
34 tasks
@bors bors bot closed this as completed in 49a3487 Mar 7, 2023
bors bot added a commit that referenced this issue Mar 8, 2023
232: Verify open-close-open DLC channel behaviour  r=luckysori a=luckysori

Actually finishes #50.

We should eventually also add assertions for the LN balances and DLC collateral values for all tests where the action results in creating a DLC.

Co-authored-by: Lucas Soriano del Pino <lucas_soriano@fastmail.com>
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 a pull request may close this issue.

1 participant