sdk: fix DLOB.updateOrder (#290)

* Fix for "readonly" bug when DLOB tries to update nodelist with new events * fix test * have updateOrder match other update order implementations * update CHANGELOG Co-authored-by: Chris Heaney <[email protected]>
drift-labs · Dec 13, 2022 · db89fda · db89fda
1 parent b3665c0
commit db89fda
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixes
 
+- ts-sdk: fix DLOB.updateOrder ([#290](https://github.com/drift-labs/protocol-v2/pull/290))
 - ts-sdk: make calculateClaimablePnl mirror on-chain logic ([#291](https://github.com/drift-labs/protocol-v2/pull/291))
 - ts-sdk: add margin trading toggle field to user accounts, update toggle margin trading function to add ability to toggle for any subaccount rather than just the active ([#285](https://github.com/drift-labs/protocol-v2/pull/285))
 

diff --git a/sdk/src/dlob/DLOB.ts b/sdk/src/dlob/DLOB.ts
@@ -270,8 +270,12 @@ export class DLOB {
 			return;
 		}
 
-		order.baseAssetAmountFilled = cumulativeBaseAssetAmountFilled;
-		this.getListForOrder(order)?.update(order, userAccount);
+		const newOrder = {
+			...order,
+		};
+		newOrder.baseAssetAmountFilled = cumulativeBaseAssetAmountFilled;
+
+		this.getListForOrder(order)?.update(newOrder, userAccount);
 
 		if (onUpdate) {
 			onUpdate();