rippled icon indicating copy to clipboard operation
rippled copied to clipboard

XLS-60d: DefaultAutoBridge

Open tequdev opened this issue 1 year ago • 1 comments

High Level Overview of Change

Changes the internal default path to include the path of the XRP bridge if sfPaths is unspecified in the Payment transaction.

Context of Change

https://github.com/XRPLF/XRPL-Standards/discussions/175

Type of Change

  • [x] New feature (non-breaking change which adds functionality)
  • [x] Tests (you added tests for code that already exists, or your new feature included in this PR)

Before / After

If relevant, use this section for an English description of the change at a technical level. If no path is specified, the impact of the transaction on the node is increased, but if a path is set to mediate XRP, the impact on the node is unaffected.

tequdev avatar Feb 20 '24 16:02 tequdev

Codecov Report

Attention: Patch coverage is 96.42857% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 76.98%. Comparing base (c88166e) to head (7e4a770).

Files Patch % Lines
src/test/app/Flow_test.cpp 95.83% 4 Missing and 1 partial :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4923      +/-   ##
===========================================
+ Coverage    76.97%   76.98%   +0.01%     
===========================================
  Files         1129     1129              
  Lines       131914   132045     +131     
  Branches     39629    39726      +97     
===========================================
+ Hits        101539   101659     +120     
- Misses       24459    24519      +60     
+ Partials      5916     5867      -49     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Mar 15 '24 07:03 codecov-commenter