[ENHANCEMENT] Improve trade history - Githubissues

DeXter-on-Radix / website

Community built order book dex on Radix.

https://dexteronradix.com/

6 stars 14 forks source link

[ENHANCEMENT] Improve trade history #366

Closed dcts closed 1 month ago

dcts commented 1 month ago

We currently have 2 problems:

TradeHistory provides no additional information compared to OrderHistory
OrderHistory also contains Open Orders, which is unexpected and confusing

My suggestion

remove tradeHistory for now
Open Order should only show orders that are pending
Order History shouly only show orders that are either cancelled or fulfilled. Pending orders are not history.

Here is a recap of all changes:

restructured to only contain "OpenOrders" and "OrderHistory"
removed "action" from OrderHistory, as order history now represents final state

(Additional changes / enhancements)

replaced "COMPLETED" with "FILLED" as its more clear to the user
de-emphasized cancelled rows by adding opacity, inspired by binance

NOTE: we should still implement "Trade Hisotry", but in a correct way, see #367 for more information.