-
Notifications
You must be signed in to change notification settings - Fork 66
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
feat: emit an event indicating that in-flight exit has finalized #647
Conversation
@@ -123,6 +127,8 @@ library PaymentProcessInFlightExit { | |||
if (!success) { | |||
emit InFlightBondReturnFailed(exit.bondOwner, exit.bondSize); | |||
} | |||
|
|||
emit InFlightExitFinalized(exitId); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whoa! how was this one missed. cool we finally cought it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we swap L141 and L142 🙈 if we add another commit
ed74974
to
8833d03
Compare
8833d03
to
9e88dd9
Compare
@kevsul @souradeep-das should we do this too? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's also generate solidoc before merge if we can?
and
#647 (comment)
Closes #646