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

Bug/exchange rates #1300

Merged
merged 5 commits into from
Aug 18, 2022
Merged

Bug/exchange rates #1300

merged 5 commits into from
Aug 18, 2022

Conversation

Tburm
Copy link
Contributor

@Tburm Tburm commented Aug 18, 2022

Fix exchange rates on swap for synths that have futures markets (sEUR)

Description

  • change the rates object in state to include synth and asset names (sEUR and EUR)
  • Update the ETH address for swap pricing to the WETH contract on Optimism

@vercel
Copy link

vercel bot commented Aug 18, 2022

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

Name Status Preview Updated
kwenta ✅ Ready (Inspect) Visit Preview Aug 18, 2022 at 7:59PM (UTC)

@Tburm Tburm requested a review from platschi August 18, 2022 17:36
@vercel vercel bot temporarily deployed to Preview August 18, 2022 17:37 Inactive
@platschi
Copy link
Collaborator

Trying ETH -> sUSD swap clicking "Max" on ETH results in 0 instead of expected max ETH balance of connected wallet

Screen Shot 2022-08-18 at 15 08 40

@vercel vercel bot temporarily deployed to Preview August 18, 2022 18:28 Inactive
@Tburm
Copy link
Contributor Author

Tburm commented Aug 18, 2022

@platschi the issue is because we "buffer" ETH swaps by 0.1 ETH which seems really extreme. Since the balance is 0.017 it will default to 0, but if you had .2 ETH it would display .1 ETH

@Tburm
Copy link
Contributor Author

Tburm commented Aug 18, 2022

const ETH_TX_BUFFER = 0.1;

@vercel vercel bot temporarily deployed to Preview August 18, 2022 19:41 Inactive
@vercel vercel bot temporarily deployed to Preview August 18, 2022 19:59 Inactive
@platschi platschi merged commit 9a0c778 into dev Aug 18, 2022
@platschi platschi deleted the bug/exchange-rates branch August 18, 2022 20:21
@LeifuChen LeifuChen mentioned this pull request Sep 5, 2022
@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.

3 participants