You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently while trying to swap MATIC to AVAX, multi-hop quotes are displayed.
However, we do not explicitly support multi-hop quotes yet.
As such, we have (wrong for multi-hop, correct for single hops) assumptions about the sell asset accountId.
These make it so we do wrong balance checks for Li.Fi multi-hops and mistakenly display "Not enough to cover gas" even when there is more than enough (see references section).
We should fix this so that these quotes are actually actionable and happy, or we filter these out.
References and additional details
Wrong accountId for multi-hop resulting in wrong balance checks:
Overview
Currently while trying to swap MATIC to AVAX, multi-hop quotes are displayed.
However, we do not explicitly support multi-hop quotes yet.
As such, we have (wrong for multi-hop, correct for single hops) assumptions about the sell asset accountId.
These make it so we do wrong balance checks for Li.Fi multi-hops and mistakenly display "Not enough to cover gas" even when there is more than enough (see references section).
We should fix this so that these quotes are actually actionable and happy, or we filter these out.
References and additional details
Wrong accountId for multi-hop resulting in wrong balance checks:
web/src/state/slices/tradeInputSlice/selectors.ts
Lines 102 to 105 in 2c882b6
Acceptance Criteria
Either:
Need By Date
No response
Screenshots/Mockups
Estimated effort
No response
The text was updated successfully, but these errors were encountered: