Skip to content

staticaddr: simplify withdrawal fsm transition #973

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

Merged
merged 1 commit into from
Jul 2, 2025

Conversation

hieblmi
Copy link
Collaborator

@hieblmi hieblmi commented Jul 1, 2025

Previously upon recovery, a withdrawing deposit was
first transitioned into the Deposited state by the
deposit manager, and then again into the Withdrawing
state by the withdrawal manager. The first transition
is unnecessary, so we just remain in the Withdrawing
state upon recovery.

@hieblmi hieblmi marked this pull request as draft July 1, 2025 11:25
@hieblmi hieblmi closed this Jul 1, 2025
@hieblmi hieblmi force-pushed the fix-withdraw-test branch from ad23568 to e145963 Compare July 1, 2025 11:59
@hieblmi hieblmi reopened this Jul 1, 2025
@hieblmi hieblmi force-pushed the fix-withdraw-test branch 8 times, most recently from 55da3c1 to 31c9649 Compare July 1, 2025 15:48
@hieblmi hieblmi closed this Jul 1, 2025
@hieblmi hieblmi force-pushed the fix-withdraw-test branch from 31c9649 to 1104e92 Compare July 1, 2025 16:17
@hieblmi hieblmi reopened this Jul 1, 2025
@hieblmi hieblmi force-pushed the fix-withdraw-test branch from 55b5671 to 1a30f90 Compare July 1, 2025 17:19
@hieblmi hieblmi closed this Jul 1, 2025
@hieblmi hieblmi force-pushed the fix-withdraw-test branch from 1a30f90 to 1104e92 Compare July 1, 2025 17:46
@hieblmi hieblmi reopened this Jul 1, 2025
@hieblmi hieblmi force-pushed the fix-withdraw-test branch 2 times, most recently from e356cc3 to a6e4409 Compare July 1, 2025 21:16
@hieblmi hieblmi closed this Jul 2, 2025
@hieblmi hieblmi force-pushed the fix-withdraw-test branch from a6e4409 to 1104e92 Compare July 2, 2025 08:20
@hieblmi hieblmi reopened this Jul 2, 2025
@hieblmi hieblmi force-pushed the fix-withdraw-test branch 2 times, most recently from c254206 to 8d7da32 Compare July 2, 2025 13:05
previously upon recovery, a withdrawing deposit was
first transitioned into the Deposited state by the
deposit manager, and then again into the Withdrawing
state by the withdrawal manager. The first transition
is unnecessary, so we just remain in the Withdrawing
state upon recovery.
@hieblmi hieblmi force-pushed the fix-withdraw-test branch from 8d7da32 to 412fe7f Compare July 2, 2025 13:42
@hieblmi hieblmi changed the title Revert "staticaddr: recover withdrawals in parallel" staticaddr: simplify withdrawal fsm transition Jul 2, 2025
@hieblmi hieblmi marked this pull request as ready for review July 2, 2025 13:47
@hieblmi hieblmi self-assigned this Jul 2, 2025
@hieblmi hieblmi requested review from starius and bhandras July 2, 2025 15:03
@hieblmi hieblmi merged commit 8464bd2 into lightninglabs:master Jul 2, 2025
4 checks passed
@hieblmi hieblmi deleted the fix-withdraw-test branch July 2, 2025 16:24
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