moontreeapp / moontree

Primary Mono Repo
6 stars 5 forks source link

Incorrect History Ordering #534

Closed Whale-Street closed 2 years ago

Whale-Street commented 2 years ago

Describe The Bug

When sending two subsequent transactions, the first transaction is being displayed second and the second first.

To Reproduce

Send a transaction, then send another. While the transactions are still in the mempool, observe them in a coin's transaction history.

Expected Behavior

They should be in the correct order by time sent.

Screenshots

Screenshot_20220504-101753_Squid

Tasks For This Bug

lastmeta commented 2 years ago

I think we order transactions by height, but maybe we have to order mempool transactions by time or something

kralverde commented 2 years ago

unfortunately mempool transactions aren't really "ordered" in any way, but we can maintain the state of what we currently think is in the mempool and add/remove from that to keep an order