Skip to content

Conversation

@ndatta-nethermind
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Sep 5, 2025

@ndatta-nethermind is attempting to deploy a commit to the dYdX Trading Team on Vercel.

A member of the Team first needs to authorize it.

@gaonip
Copy link
Contributor

gaonip commented Sep 8, 2025

As I explained before, I already pushed the change to market ID (which is correct) and not clob pair

getPrice requires the marketId and not clobPairId . These are same for most assets but can be different for some so they shouldn’t be assumed to be the same. So this issue should not be to change it back to wrong value, rather if anything, make a note to readers that this is a different ID

@ndatta-nethermind ndatta-nethermind changed the title Added clob pair id description and fixed parameter description in get_price api Added clarification on market id and perpeutal id differences Sep 9, 2025
@gaonip
Copy link
Contributor

gaonip commented Sep 10, 2025

Could you also make an exclamation note in the get price page, since this is the most noticeable place market_id will be used for users to be noticed about this

@vercel
Copy link

vercel bot commented Sep 10, 2025

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

Project Deployment Preview Comments Updated (UTC)
v4-teacher Ready Ready Preview Comment Sep 10, 2025 10:23am
vocs-docs Ready Ready Preview Comment Sep 10, 2025 10:23am

@gaonip gaonip merged commit 9e5e996 into dydxprotocol:main Sep 16, 2025
3 checks passed
ndatta-nethermind added a commit to NethermindEth/dydx-documentation that referenced this pull request Sep 17, 2025
…otocol#400)

* Added clob pair id description and fixed parameter description in get_price api

* Added clarification on market id and perpeutal id differences

* Added warning on market id and perpetual id alignment issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Keep either market id or clob pair id in description as appropriate

4 participants