-
Notifications
You must be signed in to change notification settings - Fork 20
Feature/sdk 75 support vault v2 market adapter in sdk #487
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
Feature/sdk 75 support vault v2 market adapter in sdk #487
Conversation
b93e7b0 to
0428e8a
Compare
Rubilmax
left a comment
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.
Looks great!
packages/blue-sdk-viem/contracts/vault-v2/GetVaultV2MorphoMarketV1AdapterV2.sol
Outdated
Show resolved
Hide resolved
tarikbellamine
left a comment
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.
what do you think about trying to restrict the devx of using the old adapter without removing it completely? im thinking deprecation tags, JS docs discouraging it, etc
0428e8a to
5d99239
Compare
I'm not sure about this, the sdk is read-only so we don't decide which type of adapter is being used |
Foulks-Plb
left a comment
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.
GG!
No description provided.