-
Notifications
You must be signed in to change notification settings - Fork 53
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(client): Interop consolidation sub-problem #913
Conversation
📚
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files☔ View full report in Codecov by Sentry. |
492a185
to
eb4856d
Compare
9d5594e
to
093243b
Compare
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.
Still a few todos - also beware i saw a mix of INTEROP:
and TODO:
comment prefixes used over your interop prs. Those will need to be fixed.
Thanks @refcell - those todos are left there intentionally. I'll comb through them this morning and translate to tickets if I'm not able to get to them today. |
Overview
Implements the happy-path variant of the consolidation sub-problem for the interop FPP.
This will be followed up with a larger overhaul that allows for re-executing the optimistic blocks with deposits only, in the case where dependencies were found to be invalid.