-
Notifications
You must be signed in to change notification settings - Fork 1.6k
fix DeliveredAmount
and delivered_amount
in transaction metadata for direct MPT transfer
#5569
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #5569 +/- ##
=======================================
Coverage 78.8% 78.8%
=======================================
Files 814 814
Lines 71258 71262 +4
Branches 8366 8349 -17
=======================================
+ Hits 56133 56158 +25
+ Misses 15125 15104 -21
🚀 New features to boost your workflow:
|
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 👍
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.
It looks good to me. (left a minor comment)
DeliveredAmount
and delievered_amount
in transaction metadata for direct MPT transferDeliveredAmount
and delivered_amount
in transaction metadata for direct MPT transfer
…to ximinez/lending-XLS-66 * XRPLF/ximinez/lending-refactoring-4: fix `DeliveredAmount` and `delivered_amount` in transaction metadata for direct MPT transfer (#5569)
…for direct MPT transfer (#5569) The Payment transaction metadata is missing the `DeliveredAmount` field that displays the actual amount delivered to the destination excluding transfer fees. This amendment fixes this problem.
…for direct MPT transfer (XRPLF#5569) The Payment transaction metadata is missing the `DeliveredAmount` field that displays the actual amount delivered to the destination excluding transfer fees. This amendment fixes this problem.
@shawnxie999 would you like to create a new PR to switch |
High Level Overview of Change
The Payment transaction metadata is missing the
DeliveredAmount
field that displays the actual amount delivered to the destination excluding transfer fees. This amendment fixes this problem.Context of Change
Type of Change
.gitignore
, formatting, dropping support for older tooling)API Impact
libxrpl
change (any change that may affectlibxrpl
or dependents oflibxrpl
)