Skip to content

Comments

Add Synthetix Atomic Swap support [TKR-324]#518

Merged
kyu-c merged 11 commits intodevelopmentfrom
feat/synthetix
Jul 26, 2022
Merged

Add Synthetix Atomic Swap support [TKR-324]#518
kyu-c merged 11 commits intodevelopmentfrom
feat/synthetix

Conversation

@kyu-c
Copy link
Contributor

@kyu-c kyu-c commented Jul 18, 2022

Description

  • Add Synthetix Atomic Swap support on Ethereum Mainnet and Optimism.

Testing instructions

Types of changes

Checklist:

  • Prefix PR title with [WIP] if necessary.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • Add new entries to the relevant CHANGELOG.jsons. (will be updated right before merge)

@kyu-c kyu-c requested review from dekz and dextracker as code owners July 18, 2022 20:10
@kyu-c kyu-c marked this pull request as draft July 18, 2022 20:10
@kyu-c kyu-c removed request for dekz and dextracker July 18, 2022 20:10
@kyu-c kyu-c requested a review from dekz July 18, 2022 21:19
@kyu-c kyu-c marked this pull request as ready for review July 18, 2022 21:20
// Lido
builder.addBidirectional(MAINNET_TOKENS.stETH, MAINNET_TOKENS.wstETH);
// Synthetix Atomic Swap
builder.addCompleteSubgraph([
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interested to see how the gasUsed is for this (you can console.log it in Sampler Metrics)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mainnet:

  • sUSD -> WETH (sell) : 162m
  • sUSD -> WETH (buy) : 165m
  • WETH -> sUSD (sell): 168m
  • WETH -> sUSD (buy): 158m

Optimism

  • sUSD -> WETH (sell) : 107m
  • sUSD -> WETH (buy) : 153m
  • WETH -> sUSD (sell) : 338m
  • WETH -> sUSD (buy) : 288m

@kyu-c kyu-c merged commit a5babb9 into development Jul 26, 2022
@kyu-c kyu-c deleted the feat/synthetix branch July 26, 2022 04:17
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