-
Notifications
You must be signed in to change notification settings - Fork 5
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
feat: update btc base #118
Conversation
@@ -45,7 +45,6 @@ jobs: | |||
# - run: npm run test:btc2xeth | |||
# - run: npm run test:eth2xbtc | |||
- run: npm run test:base | |||
- run: npm run test:icusd |
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.
removing as we don't need to spend time on running deprecated token
@@ -23,7 +23,8 @@ const FlashMintLeveragedAerodromeAddress = | |||
const eth = '0xEeeeeEeeeEeEeeEeEeEeeEEEeeeeEeeeeeeeEEeE' | |||
const usdcAddress = getTokenByChainAndSymbol(chainId, 'USDC').address | |||
|
|||
describe('LeveragedAerodromeBuilder()', () => { | |||
// Skip until using again with other new products | |||
describe.skip('LeveragedAerodromeBuilder()', () => { |
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.
can be skipped for now (unused)
@@ -59,7 +59,6 @@ export class FlashMintNavQuoteProvider | |||
inputToken: isMinting ? inputToken.address : usdc, | |||
outputToken: isMinting ? usdc : outputToken.address, | |||
inputAmount: inputTokenAmount.toString(), | |||
// TODO: |
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.
obsolete
} | ||
|
||
export interface FlashMintQuoteRequest { | ||
// TODO: add taker? |
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.
will probably do this when we have the new FM contracts for 0x v2 and LIFI. for now it's not necessary yet.
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.
LGTM, one small comment
Co-authored-by: 0xonramp <136734776+0xonramp@users.noreply.github.com>
🎉 This PR is included in version 3.22.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Updates BTC2X and BTC3X on Base to use new
FlashMintLeveragedMorphoAaveLM
https://basescan.org/address/0xb86E1EEf76Bc835E73B8C80eb786262C33d086D8