From dcc6c9dfbf986eb30d2cca8cfcf3527a3e548ea9 Mon Sep 17 00:00:00 2001 From: Sainan Date: Sun, 23 Feb 2025 05:45:01 +0100 Subject: [PATCH] fix: handle droptable giving a 3-day booster e.g. sorties can rarely give these --- src/services/missionInventoryUpdateService.ts | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/services/missionInventoryUpdateService.ts b/src/services/missionInventoryUpdateService.ts index 766518e2..deadb4ae 100644 --- a/src/services/missionInventoryUpdateService.ts +++ b/src/services/missionInventoryUpdateService.ts @@ -11,7 +11,6 @@ import { addFocusXpIncreases, addFusionTreasures, addGearExpByCategory, - addItem, addMiscItems, addMissionComplete, addMods, @@ -28,6 +27,7 @@ import { getEntriesUnsafe } from "@/src/utils/ts-utils"; import { IEquipmentClient } from "@/src/types/inventoryTypes/commonInventoryTypes"; import junctionRewards from "@/static/fixed_responses/junctionRewards.json"; import { IJunctionRewards } from "@/src/types/commonTypes"; +import { handleStoreItemAcquisition } from "./purchaseService"; const getRotations = (rotationCount: number): number[] => { if (rotationCount === 0) return [0]; @@ -290,8 +290,7 @@ export const addMissionRewards = async ( } for (const reward of MissionRewards) { - //TODO: additem should take in storeItems - const inventoryChange = await addItem(inventory, reward.StoreItem.replace("StoreItems/", ""), reward.ItemCount); + const inventoryChange = await handleStoreItemAcquisition(reward.StoreItem, inventory, reward.ItemCount); //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: some conditional types to rule out binchanges?