Skip to content

chore: migrate to new llamalend library #915

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

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

DanielSchiavini
Copy link
Collaborator

@DanielSchiavini DanielSchiavini commented May 7, 2025

Copy link

vercel bot commented May 7, 2025

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

Name Status Preview Comments Updated (UTC)
curve-dapp ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 7, 2025 1:36pm
curve-dapp-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 7, 2025 1:36pm

export type NetworkEnum = INetworkName
export type Provider = ethers.BrowserProvider
export type MarketListType = 'borrow' | 'supply'
export type EstimatedGas = number | number[] | null
export type OneWayMarketTemplate = LendMarketTemplate // todo: use LendMarketTemplate consistently
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Separate PR to make it easier to review

@0xAlunara
Copy link
Collaborator

Do not merge yet:

There's been a bug spotted in stablecoin api where new mint markets are incorrectly using the isEth parameter when calling remove_collateral. The same fix also needs to be applied to llamalend api.
image

@DanielSchiavini DanielSchiavini marked this pull request as draft May 7, 2025 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants