Skip to content

Conversation

@oumar-fall
Copy link
Collaborator

No description provided.

@oumar-fall oumar-fall self-assigned this Dec 16, 2025
@linear
Copy link

linear bot commented Dec 16, 2025

@oumar-fall oumar-fall force-pushed the feature/sdk-75-support-vault-v2-market-adapter-in-sdk branch 2 times, most recently from b93e7b0 to 0428e8a Compare December 17, 2025 09:56
Copy link
Collaborator

@Rubilmax Rubilmax left a comment

Choose a reason for hiding this comment

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

Looks great!

Copy link

@tarikbellamine tarikbellamine left a 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

@oumar-fall oumar-fall force-pushed the feature/sdk-75-support-vault-v2-market-adapter-in-sdk branch from 0428e8a to 5d99239 Compare December 18, 2025 11:32
@oumar-fall
Copy link
Collaborator Author

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

I'm not sure about this, the sdk is read-only so we don't decide which type of adapter is being used

@oumar-fall oumar-fall requested a review from Rubilmax December 18, 2025 13:02
Copy link
Contributor

@Foulks-Plb Foulks-Plb left a comment

Choose a reason for hiding this comment

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

GG!

@oumar-fall oumar-fall merged commit a9e05d8 into main Dec 19, 2025
17 checks passed
@oumar-fall oumar-fall deleted the feature/sdk-75-support-vault-v2-market-adapter-in-sdk branch December 19, 2025 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants