Skip to content

Support for Permissioned DEX XLS-81d#1208

Merged
ckeshava merged 6 commits intoripple:stagingfrom
ckeshava:perm_dex
Jul 25, 2025
Merged

Support for Permissioned DEX XLS-81d#1208
ckeshava merged 6 commits intoripple:stagingfrom
ckeshava:perm_dex

Conversation

@ckeshava
Copy link
Contributor

@ckeshava ckeshava commented Jul 17, 2025

High Level Overview of Change

This PR implements support for the Permissioned DEX (XLS-81D). The rippled C++ PR can be found here. This is the relevant PR for xrpl.js client library.

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)
  • Tests (You added tests for code that already exists, or your new feature included in this PR)
  • Documentation Updates
  • Translation Updates
  • Release

Codebase Modernization

  • Updated files to React Hooks
  • Updated files to TypeScript
  • Updated tests to React Testing Library
    The affected filed already make use of Typescript test frameworks (jest).

Before / After

Simple-view of the Payment transaction with DomainID. Here is the display of the currently-deployed devnet.

image

Detailed view of the Payment transaction. The above link to the currently-deployed devnet will provide you the before/after diff for this PR.
image

Detailed view of the OfferCreate transaction with the tfHybrid flag. This is the link to the transaction in the current version of the devnet-explorer.

image

Simple-view of the above OfferCreate transaction:
image

Visualization of the changes to the DirectoryNode in the transaction-metadata:
image

This PR provides support for XLS-85d Permissioned Domains in the Explorer. If DomainID field is included in the Payment and OfferCreate transactions, then this PR provides support for rendering that information.

Test Plan

Appropriate unit tests have been included.

@ckeshava ckeshava requested review from Patel-Raj11 and pdp2121 July 17, 2025 02:49
Copy link
Contributor

@Patel-Raj11 Patel-Raj11 left a comment

Choose a reason for hiding this comment

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

LGTM!

@pdp2121
Copy link
Collaborator

pdp2121 commented Jul 21, 2025

@ckeshava could you attach screenshots of the pages that was added/modified in this PR?
Similar to this:
#1199

@ckeshava
Copy link
Contributor Author

@ckeshava could you attach screenshots of the pages that was added/modified in this PR? Similar to this: #1199

@pdp2121 I have updated the description of the PR

"offer_create_desc_line_1": "<1><0>{{account}}</0></1> は、<5><0>{{takerPays}}</0><1><0>{{currency}}</0></1></5> を取得するために、<3><0>{{takerGets}}</0><1><0>{{currency}}</0></1></3> を支払うオファーを作成しました。",
"offer_create_desc_line_2": "このオファーの変換レートは",
"offer_create_desc_line_3": "トランザクションによって、アカウントの次のオファー番号も取り消されます :",
"offer_create_desc_line_5": "このオファーは許可ドメインに適用されます",
Copy link
Collaborator

Choose a reason for hiding this comment

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

You don't have to input translation for languages other than English. Can just put null and specialized community contributors will fill them in later

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, I'll that iin mind for the next feature. However, for this PR, I verified that all the translations are accurate (with Google Translate) software.

Do you still want to remove them? I'd prefer to keep them.

</span>
)}
{fields.DomainID && (
<span> pertaining to the Permissioned Domain: {fields.DomainID}</span>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Needs translation hook for this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

addressed in 78927b4

@ckeshava ckeshava requested a review from pdp2121 July 24, 2025 20:11
@ckeshava ckeshava merged commit 3cbc982 into ripple:staging Jul 25, 2025
5 checks passed
@ckeshava
Copy link
Contributor Author

Thank you for the reviews @pdp2121 @Patel-Raj11

pdp2121 pushed a commit that referenced this pull request Oct 8, 2025
This PR implements support for the [Permissioned DEX
(XLS-81D)](https://github.com/XRPLF/XRPL-Standards/blob/56ba122be9f127c940fd37d433faaa2de91a4eb6/XLS-0081d-permissioned-dex/README.md).
The rippled C++ PR can be [found
here](https://github.com/XRPLF/rippled/pull/5404/files). This is [the
relevant PR](XRPLF/xrpl.js#3032) for xrpl.js
client library.
pdp2121 pushed a commit that referenced this pull request Oct 8, 2025
This PR implements support for the [Permissioned DEX
(XLS-81D)](https://github.com/XRPLF/XRPL-Standards/blob/56ba122be9f127c940fd37d433faaa2de91a4eb6/XLS-0081d-permissioned-dex/README.md).
The rippled C++ PR can be [found
here](https://github.com/XRPLF/rippled/pull/5404/files). This is [the
relevant PR](XRPLF/xrpl.js#3032) for xrpl.js
client library.
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.

3 participants