Skip to content
This repository was archived by the owner on Oct 28, 2025. It is now read-only.

Conversation

@0xsambugs
Copy link
Contributor

We are now updating the OKX dex source to reflect the changes to their API

@0xsambugs 0xsambugs self-assigned this May 9, 2025
@0xsambugs 0xsambugs requested a review from 0xKoaj May 9, 2025 14:44
}: QuoteParams<OKXDexSupport, OKXDexConfig>) {
const queryParams = {
chainId,
chainIndex: chainId,
Copy link
Contributor

Choose a reason for hiding this comment

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

WHY? xD

Copy link
Contributor Author

Choose a reason for hiding this comment

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

🤷

Copy link
Contributor

@0xKoaj 0xKoaj left a comment

Choose a reason for hiding this comment

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

Great work!

@0xsambugs 0xsambugs merged commit ace242a into main May 9, 2025
3 checks passed
@0xsambugs 0xsambugs deleted the fix/okx-dex-changes branch May 9, 2025 14:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants