-
Notifications
You must be signed in to change notification settings - Fork 232
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(#minor); gmx; fix position snapshots #2241
Conversation
passed QA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Just 1 extra comment, can you copy all of the sdk changes over to the gmx-forks repo so we don't have some files that show the old version.
Thanks!
Following discussion here: https://discord.com/channels/953684103012683796/1067631209968058408/1126475904944705626 do you think you can add Also update the GMX updates you have with this? If not, no worries, i can add it after this is merged. |
@ishraq8 added, and updated deployment links in the description |
updates to |
@dhruv-chauhan can we also update versions for |
@dmelotik done |
Deployments,
avalanche: https://okgraph.xyz/?q=Qmemei77paarcRzt1exCzcNuBJhjTmeRsLHSAxDGTLjfxR
arbitrum: https://okgraph.xyz/?q=Qmf1vnAzQS4SCgy3nzEy1m1X5iGgbqMobGPJtKU8E4o7v3
Example query,
https://api.thegraph.com/subgraphs/id/Qmemei77paarcRzt1exCzcNuBJhjTmeRsLHSAxDGTLjfxR/graphql?query=%7B%0A++positions+%7B+%0A++++id%0A++++timestampOpened%0A++++hashOpened%0A++++timestampClosed%0A++++hashClosed%0A++++snapshots%28orderBy%3A+timestamp%2C+orderDirection%3A+asc%29+%7B+%0A++++++balance+%0A++++++balanceUSD+%0A++++++collateralBalance+%0A++++++collateralBalanceUSD+%0A++++++timestamp+%0A++++++hash%0A++++%7D+%0A++%7D%0A%7D