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 #5183

Merged
merged 21 commits into from
Nov 7, 2024
Merged

Proposed 2.3.0-rc1 #5183

merged 21 commits into from
Nov 7, 2024

Conversation

ximinez
Copy link
Collaborator

@ximinez ximinez commented Nov 7, 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.
@ximinez ximinez changed the base branch from develop to release November 7, 2024 00:59
@ximinez ximinez mentioned this pull request Nov 7, 2024
3 tasks
@ximinez ximinez added the Passed Passed code review & PR owner thinks it's ready to merge. Perf sign-off may still be required. label Nov 7, 2024
@ximinez ximinez merged commit 9d58f11 into release Nov 7, 2024
53 of 54 checks passed
@ximinez ximinez deleted the release-next branch November 7, 2024 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Passed Passed code review & PR owner thinks it's ready to merge. Perf sign-off may still be required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.