OpenWF/SpaceNinjaServer
12
6
Fork 15
Code Issues 46 Pull Requests 6 Actions Packages Projects Releases Activity

fix: ignore purchaseQuantity when giving mission rewards (#1446)
Some checks failed
Build Docker image / docker (push) Waiting to run
Details
Build / build (18) (push) Has been cancelled
Details
Build / build (20) (push) Has been cancelled
Details
Build / build (22) (push) Has been cancelled
Details

Browse Source 
Reviewed-on: #1446
Co-authored-by: Sainan <63328889+Sainan@users.noreply.github.com>
Co-committed-by: Sainan <63328889+Sainan@users.noreply.github.com>
This commit is contained in:
Sainan 2025-04-03 10:37:52 -07:00 committed by Sainan
parent 92cf85084f
commit 05c0c9909c
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/services/missionInventoryUpdateService.ts
View File

@ -537,7 +537,13 @@ export const addMissionRewards = async (
} }
for (const reward of MissionRewards) { for (const reward of MissionRewards) {
const inventoryChange = await handleStoreItemAcquisition(reward.StoreItem, inventory, reward.ItemCount); const inventoryChange = await handleStoreItemAcquisition(
reward.StoreItem,
inventory,
reward.ItemCount,
undefined,
true
);
//TODO: combineInventoryChanges improve type safety, merging 2 of the same item? //TODO: combineInventoryChanges improve type safety, merging 2 of the same item?
//TODO: check for the case when two of the same item are added, combineInventoryChanges should merge them, but the client also merges them //TODO: check for the case when two of the same item are added, combineInventoryChanges should merge them, but the client also merges them
//TODO: some conditional types to rule out binchanges? //TODO: some conditional types to rule out binchanges?