Skip to content

Conversation

shawnxie999
Copy link
Collaborator

@shawnxie999 shawnxie999 commented Jul 16, 2025

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Performance (increase or change in throughput and/or latency)
  • Tests (you added tests for code that already exists, or your new feature included in this PR)
  • Documentation update
  • Chore (no impact to binary, e.g. .gitignore, formatting, dropping support for older tooling)
  • Release

API Impact

  • Public API: New feature (new methods and/or new fields)
  • Public API: Breaking change (in general, breaking changes should only impact the next api_version)
  • libxrpl change (any change that may affect libxrpl or dependents of libxrpl)
  • Peer protocol change (must be backward compatible or bump the peer protocol version)

@shawnxie999 shawnxie999 marked this pull request as ready for review July 16, 2025 15:29
@shawnxie999 shawnxie999 requested a review from a team as a code owner July 16, 2025 15:30
@shawnxie999 shawnxie999 linked an issue Jul 16, 2025 that may be closed by this pull request
Copy link

codecov bot commented Jul 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.8%. Comparing base (9b45b68) to head (5db6318).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           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     
Files with missing lines Coverage Δ
src/xrpld/app/tx/detail/Payment.cpp 93.8% <100.0%> (+0.1%) ⬆️

... and 9 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Collaborator

@gregtatcam gregtatcam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@kennyzlei kennyzlei requested a review from yinyiqian1 July 25, 2025 17:25
Copy link
Collaborator

@yinyiqian1 yinyiqian1 left a 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)

@shawnxie999 shawnxie999 added the Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. label Jul 29, 2025
@bthomee bthomee changed the title fix DeliveredAmount and delievered_amount in transaction metadata for direct MPT transfer fix DeliveredAmount and delivered_amount in transaction metadata for direct MPT transfer Jul 29, 2025
@bthomee bthomee enabled auto-merge (squash) July 29, 2025 16:03
@bthomee bthomee merged commit baf4b83 into XRPLF:develop Jul 29, 2025
25 checks passed
ximinez added a commit that referenced this pull request Jul 30, 2025
…to ximinez/lending-XLS-66

* XRPLF/ximinez/lending-refactoring-4:
  fix `DeliveredAmount` and `delivered_amount` in transaction metadata for direct MPT transfer (#5569)
bthomee pushed a commit that referenced this pull request Jul 31, 2025
…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.
mvadari pushed a commit to mvadari/rippled that referenced this pull request Aug 5, 2025
…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.
@legleux legleux mentioned this pull request Aug 27, 2025
@Bronek
Copy link
Collaborator

Bronek commented Sep 1, 2025

@shawnxie999 would you like to create a new PR to switch Supported::no to yes for this amendment, before the 3.0.0 release ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Amendment Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MPT Delivered Amount Transfer Fee (Version: 2.5.0)

5 participants