Skip to content
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

Proposed 2.3.0-rc1 #5182

Closed
wants to merge 21 commits into from
Closed

Proposed 2.3.0-rc1 #5182

wants to merge 21 commits into from

Conversation

ximinez
Copy link
Collaborator

@ximinez ximinez commented Nov 6, 2024

High Level Overview of Change

This is a release candidate for the 2.3.0 release.

Highlights:

The base branch is release. All releases (including betas) go in release. This PR will be merged with --ff-only (not squashed or rebased, and not using the GitHub UI) to both release and develop.

Context of Change

This introduces

  • the Credentials amendment to support on-ledger credentials.
  • the MPTokensV1 amendment to support Multi-purpose tokens.
  • the AMMClawback amendment to add support for clawing back tokens held in an AMM.
  • the fixAMMv1_2 amendment to fix some issues in AMM functionality.

Type of Change

  • Release

API Impact

  • Public API: New feature (new methods and/or new fields)
  • libxrpl change (any change that may affect libxrpl or dependents of libxrpl)

intelliot and others added 21 commits October 1, 2024 14:07
Move the newest information to the top, i.e., use reverse chronological order within each of the two sections ("API Versions" and "XRP Ledger server versions")
Validator operators have been confused by the rpcInternal error, which can occur if the server is not running in another process.
Amendment:
- MPTokensV1

New Transactions:
- MPTokenIssuanceCreate
- MPTokenIssuanceDestroy
- MPTokenIssuanceSet
- MPTokenAuthorize

Modified Transactions:
- Payment
- Clawback

New Objects:
- MPTokenIssuance
- MPToken

API updates:
- ledger_entry
- account_objects
- ledger_data

Other:
- Add += and -= operators to ValueProxy

Read full spec: https://github.com/XRPLF/XRPL-Standards/tree/master/XLS-0033d-multi-purpose-tokens

---------
Co-authored-by: Shawn Xie <[email protected]>
Co-authored-by: Howard Hinnant <[email protected]>
Co-authored-by: Ed Hennis <[email protected]>
Co-authored-by: John Freeman <[email protected]>
fix the typo "concensus" -> "consensus"
Amendment:
- AMMClawback

New Transactions:
- AMMClawback

Modified Transactions:
- AMMCreate
- AMMDeposit
* Add reserve check on AMM Withdraw
* Try AMM max offer if changeSpotPriceQuality() fails
* Checks only Currency or MPT Issuance ID part of the Asset object.
* Resolves temREDUNDANT regression detected in testing.
    Amendment:
    - Credentials
    
    New Transactions:
    - CredentialCreate
    - CredentialAccept
    - CredentialDelete
    
    Modified Transactions:
    - DepositPreauth
    - Payment
    - EscrowFinish
    - PaymentChannelClaim
    - AccountDelete
    
    New Object:
    - Credential

    Modified Object:
    - DepositPreauth
    
    API updates:
    - ledger_entry
    - account_objects
    - ledger_data
    - deposit_authorized
    
    Read full spec: https://github.com/XRPLF/XRPL-Standards/tree/master/XLS-0070d-credentials
* 2.2.2 changed functions acquireAsync and NetworkOPsImp::recvValidation to add an item to a collection under lock, unlock, do some work, then lock again to do remove the item. It will deadlock if an exception is thrown while adding the item - before unlocking.
* Replace ScopedUnlock with scope_unlock.
@kennyzlei
Copy link
Collaborator

@godexsoft can you check "libXRPL compatibility with Clio / Publish libXRPL" and verify if there is a Clio compatibility issue?

@kennyzlei kennyzlei requested a review from godexsoft November 6, 2024 23:02
Copy link

codecov bot commented Nov 6, 2024

Codecov Report

Attention: Patch coverage is 92.99553% with 47 lines in your changes missing coverage. Please review.

Project coverage is 77.9%. Comparing base (1fbf8da) to head (9d58f11).
Report is 21 commits behind head on release.

Files with missing lines Patch % Lines
src/libxrpl/protocol/STAmount.cpp 88.3% 20 Missing ⚠️
src/libxrpl/protocol/STParsedJSON.cpp 64.7% 6 Missing ⚠️
src/libxrpl/protocol/MPTIssue.cpp 86.7% 4 Missing ⚠️
src/libxrpl/protocol/Rate2.cpp 50.0% 3 Missing ⚠️
include/xrpl/protocol/Asset.h 95.0% 2 Missing ⚠️
src/libxrpl/basics/MPTAmount.cpp 87.5% 2 Missing ⚠️
src/libxrpl/beast/insight/NullCollector.cpp 50.0% 2 Missing ⚠️
src/libxrpl/protocol/Asset.cpp 91.3% 2 Missing ⚠️
src/libxrpl/protocol/STObject.cpp 0.0% 2 Missing ⚠️
src/libxrpl/beast/utility/src/beast_Journal.cpp 50.0% 1 Missing ⚠️
... and 3 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           release   #5182     +/-   ##
=========================================
+ Coverage     76.2%   77.9%   +1.7%     
=========================================
  Files          760     782     +22     
  Lines        61568   66616   +5048     
  Branches      8126    8161     +35     
=========================================
+ Hits         46909   51902   +4993     
- Misses       14659   14714     +55     
Files with missing lines Coverage Δ
include/xrpl/basics/Expected.h 100.0% <100.0%> (ø)
include/xrpl/basics/FeeUnits.h 90.4% <100.0%> (+0.3%) ⬆️
include/xrpl/basics/IOUAmount.h 100.0% <100.0%> (ø)
include/xrpl/basics/MPTAmount.h 100.0% <100.0%> (ø)
include/xrpl/basics/Number.h 100.0% <100.0%> (ø)
include/xrpl/basics/XRPAmount.h 98.8% <100.0%> (+<0.1%) ⬆️
include/xrpl/basics/base_uint.h 96.8% <100.0%> (+0.1%) ⬆️
include/xrpl/basics/safe_cast.h 81.8% <ø> (ø)
include/xrpl/basics/scope.h 100.0% <100.0%> (ø)
include/xrpl/basics/tagged_integer.h 100.0% <100.0%> (ø)
... and 153 more

... and 444 files with indirect coverage changes

Impacted file tree graph

@kennyzlei
Copy link
Collaborator

@godexsoft can you check "libXRPL compatibility with Clio / Publish libXRPL" and verify if there is a Clio compatibility issue?

Rerunning "Check libXRPL compatibility with Clio / Publish libXRP" succeeded

@ximinez
Copy link
Collaborator Author

ximinez commented Nov 7, 2024

Superseded by #5183

@ximinez ximinez closed this Nov 7, 2024
@ximinez ximinez deleted the release-next branch November 7, 2024 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.