Ellipsis-Labs / phoenix-sdk

SDK for interacting with the Phoenix DEX.
83 stars 39 forks source link

fix fill summary indexing bug #136

Closed RahulJain28 closed 12 months ago

eugene-chen commented 12 months ago

bump version

need TS too, but can be different PR

eugene-chen commented 12 months ago

Also, upstream of here, wherever we initialize this variable we should verify it's in the state we expect it to be

RahulJain28 commented 12 months ago

TS shouldn't need an update, the current code doesn't track the direction

export const fillSummaryEventBeet = new beet.BeetArgsStruct<FillSummaryEvent>(
  [
    ["index", beet.u16],
    ["clientOrderId", beet.u128],
    ["totalBaseLotsFilled", beet.u64],
    ["totalQuoteLotsFilled", beet.u64],
    ["totalFeeInQuoteLots", beet.u64],
  ],
  "FillSummaryEvent"
);