Implement wICX

[hetfly]

close #1534

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
balanced-network-interface	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 8, 2025 8:56am
balanced-network-stats	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 8, 2025 8:56am
hanaswap	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 8, 2025 8:56am
hswap	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 8, 2025 8:56am

[hetfly]

@FezBox managed to temporarily put in a check for a better trade using the queue while there is still liquidity. We will have to do a clean up after the queue is emptied and deprecated anyway, there is lot's of conditional code using the queue.

Can we get some liquidity in the new pool so that we can properly test the routing and trade picking? (queue vs wICX)

@parrot9design only when supplying and trading into ICX, wICX contract is used, but it's all abstracted from the user and ICX symbol is displayed instead of wICX. While we have the queue working in parallel temporarily, do we want to differentiate the wICX contract somehow? At least in Your pools section, we could show wICX/sICX as the pool name? If the user had both, the queue and wICX/sICX in that section, it could look a bit confusing.

[parrot9design]

Let's use ICX / sICX to identify the new pool so we can omit all mention of wICX from the UI.

We can call the old one the ICX queue:

[parrot9design]

There are still a few mentions of wICX in the UI:

The asset selectors treat ICX as though it's wICX, as it should be above sICX when the list is ordered alphabetically.

When you choose ICX, can we make the second asset default to sICX instead of bnUSD?

Also, shouldn't we be able to stake LP tokens for the new pool?