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

feat: update btc base #118

Merged
merged 37 commits into from
Feb 26, 2025
Merged

feat: update btc base #118

merged 37 commits into from
Feb 26, 2025

Conversation

janndriessen
Copy link
Collaborator

@janndriessen janndriessen commented Feb 4, 2025

Updates BTC2X and BTC3X on Base to use new FlashMintLeveragedMorphoAaveLM

https://basescan.org/address/0xb86E1EEf76Bc835E73B8C80eb786262C33d086D8

@@ -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
Copy link
Collaborator Author

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()', () => {
Copy link
Collaborator Author

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:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

obsolete

@janndriessen janndriessen marked this pull request as ready for review February 25, 2025 10:52
}

export interface FlashMintQuoteRequest {
// TODO: add taker?
Copy link
Collaborator Author

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.

Copy link
Collaborator

@0xonramp 0xonramp left a 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>
@janndriessen janndriessen merged commit 48ee1fc into main Feb 26, 2025
2 checks passed
@janndriessen janndriessen deleted the task/update-btc-base branch February 26, 2025 06:06
Copy link

🎉 This PR is included in version 3.22.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants