-
Notifications
You must be signed in to change notification settings - Fork 147
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
Defragmentation Memos #3170
Defragmentation Memos #3170
Conversation
✅ Good job, full-service was built successfully. |
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.
This PR followed the style of other memos and most of my comments are mainly suggestions to move away from that style.
transaction/builder/src/memo_builder/defragmentation_memo_builder.rs
Outdated
Show resolved
Hide resolved
transaction/builder/src/memo_builder/defragmentation_memo_builder.rs
Outdated
Show resolved
Hide resolved
transaction/builder/src/memo_builder/defragmentation_memo_builder.rs
Outdated
Show resolved
Hide resolved
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.
This looks good to go to me, so long as the issues @nick-mobilecoin has already identified are resolved.
…der.rs Co-authored-by: Nick Santana <nick@mobilecoin.com>
Co-authored-by: Nick Santana <nick@mobilecoin.com>
Co-authored-by: Nick Santana <nick@mobilecoin.com>
…der.rs Co-authored-by: Nick Santana <nick@mobilecoin.com>
…der.rs Co-authored-by: Nick Santana <nick@mobilecoin.com>
Co-authored-by: Nick Santana <nick@mobilecoin.com>
Co-authored-by: Nick Santana <nick@mobilecoin.com>
Co-authored-by: Nick Santana <nick@mobilecoin.com>
Co-authored-by: Nick Santana <nick@mobilecoin.com>
✅ Good job, full-service was built successfully. |
Co-authored-by: Nick Santana <nick@mobilecoin.com>
…der.rs Co-authored-by: Nick Santana <nick@mobilecoin.com>
…der.rs Co-authored-by: Nick Santana <nick@mobilecoin.com>
Co-authored-by: Nick Santana <nick@mobilecoin.com>
Co-authored-by: Nick Santana <nick@mobilecoin.com>
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.
LGTM Thanks!
Motivation
This PR implements changes outlined in MCIP #61.
MCIP #61 proposes a new memo type: the defragmentation memo. This memo type is used to denote defragmentation transactions. It also includes an ID field that can be used to easily group together multiple rounds of one defragmentation session together for the purpose of reporting total paid in fees and total amount defragmented. The full implementation details and rationale for defragmentation memos can be found here.