-
Notifications
You must be signed in to change notification settings - Fork 44
feat(docs): swap USDT0 to FXRP using Uniswap V3 guide #976
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
Conversation
|
|
||
| // 1. Defines Uniswap V3 router functions for swaps | ||
| interface ISwapRouter { | ||
| struct ExactInputSingleParams { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are you defining the interfaces here? Don't they have an official package that you can import from?
...
I found this: https://www.npmjs.com/package/@uniswap/v3-periphery?activeTab=code .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We talked with @jO-Osko that I will add a link to it, but not include it in the starter kits.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then add a :::note field somewhere in the guide, explicitly stating that you have defined the structs here, but that they should use the official library.
78970df to
8e99ada
Compare
| await deployAndVerifyContract(); | ||
|
|
||
| const { deployer, usdt0, fxrp, initialUsdt0Balance, initialFxrpBalance } = | ||
| await setupAndInitializeTokens(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
setUpAndInitializeTokens
|
|
||
| ## Required Addresses on Flare Mainnet | ||
|
|
||
| - V3 SwapRouter (SparkDEX): [0x8a1E35F5c98C4E85B36B7B253222eE17773b2781](https://flarescan.com/address/0x8a1E35F5c98C4E85B36B7B253222eE17773b2781/contract/14/code). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've been defaulting to Flare Explorer, not Flarescan.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sadly, it is not verified on Flare Explorer.
…AssetManager import
8e99ada to
3c750d2
Compare
…izeTokens for consistency
No description provided.