Skip to content

feat(sdk): dex#542

Open
R0bi7 wants to merge 24 commits intomainfrom
feat/sdk-dex
Open

feat(sdk): dex#542
R0bi7 wants to merge 24 commits intomainfrom
feat/sdk-dex

Conversation

@R0bi7
Copy link
Copy Markdown
Contributor

@R0bi7 R0bi7 commented Nov 19, 2025

Description

This PR closes Add dex to SDK
#213.

Checklist

  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines

@vercel
Copy link
Copy Markdown

vercel bot commented Nov 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sodax-demo Ready Ready Preview, Comment Mar 11, 2026 10:41am
sodax-web Ready Ready Preview, Comment Mar 11, 2026 10:41am

Request Review

@AntonAndell
Copy link
Copy Markdown
Contributor

Maybe we clarify what the depositing token does in demo as well so it's easier to understand.

There is also a issue with xSODA i think that is not very consistent and currently it crashes

@AntonAndell
Copy link
Copy Markdown
Contributor

for Sonic i can't approve wETH to deposit

…at/sdk-dex

# Conflicts:
#	apps/demo/src/components/shared/header.tsx
#	packages/sdk/src/moneyMarket/MoneyMarketService.ts
#	packages/sdk/src/shared/services/index.ts
#	packages/sdk/src/shared/types.ts
#	packages/sdk/src/swap/SwapService.ts
#	packages/types/src/common/index.ts
#	pnpm-lock.yaml
…at/sdk-dex

# Conflicts:
#	apps/demo/src/App.tsx
#	apps/demo/src/components/shared/header.tsx
#	packages/sdk/src/index.ts
#	packages/sdk/src/shared/constants.ts
#	packages/sdk/src/shared/entities/Sodax.ts
#	packages/sdk/src/shared/guards.ts
#	packages/sdk/src/shared/types.ts
#	pnpm-lock.yaml
…at/sdk-dex

# Conflicts:
#	apps/demo/src/lib/utils.ts
#	pnpm-lock.yaml
…at/sdk-dex

# Conflicts:
#	packages/sdk/src/shared/guards.ts
#	packages/sdk/src/shared/types.ts
#	packages/types/src/common/index.ts
#	pnpm-lock.yaml
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.

2 participants