XRPL-Labs / Xaman-Issue-Tracker

Bugs, improvements, suggestions & release progress (Project boards)
https://xumm.app
14 stars 9 forks source link

[Major change / Improvement] Change sending flow, UX + allow all paths #129

Closed WietseWind closed 3 years ago

WietseWind commented 4 years ago

Suggested new Send flow, to streamline sending any currency to any currency, taking care of paths, rates (DEX), & 'wizard' based sending. Allow to send anything and deliver anything. Allow the user to select what to SEND and what to DELIVER, with:

To consider:

Idea: to make this less intimidating to users: we could just allow users to enter the sending amount at first, then to just inform them what will be delivered (existing behaviour; xrp<>xrp, same IOU<>same IOU, or sending IOU> XRP. The user could then tap "Change" and go the the more advanced behaviour where they can select SEND/DELIVER and pick currencies.

Use ... to find the available options & rates:

image

Impacts:

18 Destination tag handling

109 IOU DEX exchange notification

76 Cross IOU sending

❗ When doing this make over, we should implement the same logic for:

canny[bot] commented 4 years ago

This issue has been linked to a Canny post: XRPL PathFinding: Currency conversions. :tada:

WietseWind commented 4 years ago

Note: pretty cool @N3TC4T:

{
  "command": "path_find",
  "subcommand": "create",
  "source_account": "rPdvC6ccq8hCdPKSPJkPmyZ4Mi1oG2FFkT",
  "destination_account": "rwiETSee2wMz3SBnAG8hkMsCgvGy9LWbZ1",
  "destination_amount": {
    "value": "100",
    "currency": "USD",
    "issuer": "rvYAfWj5gh67oV6fW32ZzP3Aw4Eubs59B"
  }
}

» image

WietseWind commented 3 years ago

Duplicates:

129

76

108

Will be implemented as xApp (anything to anything)