[PFT-1796] Add changes for the snx subgraph code and order #75
Conversation
| @@ -73,67 +24,18 @@ export const fetchAllMarketsDataQuery = gql` | |||
| export const fetchMarketByIdQuery = (marketId: string) => gql` | |||
| { | |||
| market(id: "${marketId.toLowerCase()}") { | |||
There was a problem hiding this comment.
Have we tested this? The market id in the latest snx subgraph is a number (even though IDs are stored as strings). Please check the subgraph.yaml and the returned data.
Also update the test cases
There was a problem hiding this comment.
@Akhilleshgoswami Not sure if anything was changed here. Could you please explain?
There was a problem hiding this comment.
so I have only written test case for the order, positions, and account as we were using the market from the synthetix parifi SDK , I didn't change the code here
… snx-subgraph-code
There was a problem hiding this comment.
@iamamitkumar0512 @Akhilleshgoswami Please work on the requested changes
| @@ -190,3 +193,21 @@ export const fetchPositionIdsForOrderIds = (orderIds: string[]) => gql` | |||
| } | |||
| } | |||
| `; | |||
|
|
|||
| export const fectchCollateralForOrderUsingAccountId = (accountId: string | string[]) => gql` | |||
| @@ -1,61 +1,12 @@ | |||
| import { getParifiSdkInstanceForTesting } from '..'; | |||
There was a problem hiding this comment.
Account => Wallet
SnxAccount is a different term. Please do not mix them up
No description provided.