Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disabled no synth card #1355

Merged
merged 6 commits into from
Sep 5, 2022
Merged

Disabled no synth card #1355

merged 6 commits into from
Sep 5, 2022

Conversation

LeifuChen
Copy link
Contributor

@LeifuChen LeifuChen commented Sep 5, 2022

Description

  1. The current exchange page will show You have no open positions. Open a position now: in exchange page when users have no synthetic assets in their wallet, which is not fit as exchange page supports erc20 swap right now.
    image

2.ETH => sUSD throws the error after the change in #1300.
The root cause is that 1inch and coingecko use the different token addresses to represent ETH.

  • 1inch API: ETH uses the same token address on ehtereum mainnet although this address is not pointing to any token on optimism
  • Coingecko: ETH uses the WETH's token address to show the price.

Related issue

Motivation and Context

Improve the UX with newcomers who have no synthetic assets.

How Has This Been Tested?

  1. Use a test wallet without synthetic assets and ETH.
  2. Swap ETH to sUSD on the exchange page and see the prompt No insufficient balance

Screenshots (if appropriate):

@vercel
Copy link

vercel bot commented Sep 5, 2022

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

Name Status Preview Updated
kwenta ✅ Ready (Inspect) Visit Preview Sep 5, 2022 at 9:00PM (UTC)

@LeifuChen LeifuChen changed the base branch from dev to main September 5, 2022 15:59
@vercel vercel bot temporarily deployed to Preview September 5, 2022 16:01 Inactive
@vercel vercel bot temporarily deployed to Preview September 5, 2022 16:13 Inactive
@vercel vercel bot temporarily deployed to Preview September 5, 2022 18:13 Inactive
@vercel vercel bot temporarily deployed to Preview September 5, 2022 19:10 Inactive
@vercel vercel bot temporarily deployed to Preview September 5, 2022 19:20 Inactive
@vercel vercel bot temporarily deployed to Preview September 5, 2022 21:00 Inactive
@LeifuChen LeifuChen marked this pull request as ready for review September 5, 2022 21:04
@platschi platschi self-requested a review September 5, 2022 21:10
@platschi platschi merged commit 337472f into main Sep 5, 2022
platschi pushed a commit that referenced this pull request Sep 5, 2022
* Disabled no synth card
* Added ETH approval
* Limit the liquidity source
* Update the token address
* Fixed the lint error and update the token address
* Fix the swap pricing display
@platschi platschi deleted the fix/exchange-onboard-message branch September 29, 2022 15:42
@KngZhi KngZhi mentioned this pull request Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants