search

XRPLF / rippled

Decentralized cryptocurrency blockchain daemon implementing the XRP Ledger protocol in C++
https://xrpl.org
ISC License
4.54k stars 1.47k forks source link

Convert currency during CheckCash. #4464

Open tequdev opened 1 year ago

tequdev commented 1 year ago

CheckCach transactions can only be cashed in the currency that matches the SendMax of the currently corresponding Check object.

The value of the XRPL would be enhanced if the sender of the CheckCach transaction (the destination of the Check) could receive it in any currency via the XRP Ledger's DEX.

scottschurr commented 1 year ago

I'm not weighing in on whether or not this is a good idea, but the in-ledger implementation would not be difficult. If an optional Paths field were added to CheckCash then the transaction could handle the Paths field much like a Payment does today. The change would, of course, still require an amendment.

But ease of implementation does not provide necessary justification for a change. Checks are currently a little-used feature of the ledger. Is there agreement that a change like this would significantly improve the utility of Checks?

tequdev commented 1 year ago

I know that Check is currently not in much use, and I am not sure if this change will significantly expand the use cases for XRP Ledger.

However, I think if XRP Ledger's built-in DEX could be used for a wider variety of transactions, it would increase the value of XRP Ledger itself.

It is just a low priority proposal.