WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@0xtygra
Copy link

@0xtygra 0xtygra commented Dec 11, 2025

While not shown in the API docs, in order to get the metaAndAssetCtxs for non default dexs (ie: xyz HIP-3 assets) you can and must pass a "dex" argument in the POST body.

This PR adds that to the type for "metaAndAssetCtxs". The default for the API is an empty string which represents Hyperliquid's default dex itself.

NOTE: this does slightly change the signature such that you need to call .meta_and_asset_contexts(None) once this merges, so it is a breaking change. I'm also open/happy to implement a new method .meta_and_asset_contexts_with_dex() to make it non-breaking

image

@0xtygra
Copy link
Author

0xtygra commented Dec 11, 2025

Thinking through this a bit more - websockets need this as well. One other option is to not add a dex param but instead, if the coin has a colon in it pass in the split(":")[0] as the dex as what precedes the colon is the dex name. Happy to make this change too and make it for websockets/all info endpoints that can take a dex

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.

1 participant