Skip to content

Conversation

yeager-eren
Copy link
Collaborator

@yeager-eren yeager-eren commented Jun 14, 2025

Summary

Adding XRPL types to the supported blockchains.

Currently, Our server only returning Payment, but I've added TrustSet as well since client needs to create that transaction itself (it's similar to EVM approval).

The server will return XRPL.js types as data in its response, So I copy pasted the type from that lib. The reason I didn't add the lib as dependency is it will increase the package size and we only need some typings, I also couldn't add to devDependencies since the types are publishing and exists in output, so it will not be installed on developer's machine.

part of rango-exchange/rango-client#1096

Copy link
Contributor

@RyukTheCoder RyukTheCoder left a comment

Choose a reason for hiding this comment

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

Just added a comment.

Copy link
Contributor

@RyukTheCoder RyukTheCoder left a comment

Choose a reason for hiding this comment

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

LGTM

@yeager-eren yeager-eren merged commit 1f3a236 into main Jun 16, 2025
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.

2 participants