[CharInv] Preserve pre-find-and-replace status changes in goal history (#4041)

Author: imnasnainaec

src/goals/CharacterInventory/Redux/CharacterInventoryActions.ts

@@ -118,9 +118,7 @@ export function uploadInventory() {
         validCharacters: charInvState.validCharacters,
       })
     );
-    const changes = getState().goalsState.currentGoal.changes as CharInvChanges;
-    dispatch(addCharInvChangesToGoal({ ...changes, charChanges }));
-    await dispatch(asyncUpdateGoal());
+    await dispatch(addCharChanges(charChanges));
   };
 }
 
@@ -172,6 +170,20 @@ export function loadCharInvData() {
   };
 }
 
+/** Returns a dispatch function to: in both frontend and backend, add to the current
+ * goal's changes new character inventory status changes. */
+function addCharChanges(charChanges: CharacterChange[]) {
+  return async (dispatch: StoreStateDispatch, getState: () => StoreState) => {
+    const changes: CharInvChanges = {
+      ...defaultCharInvChanges,
+      ...getState().goalsState.currentGoal.changes,
+    };
+    changes.charChanges = [...changes.charChanges, ...charChanges];
+    dispatch(addCharInvChangesToGoal(changes));
+    await dispatch(asyncUpdateGoal());
+  };
+}
+
 /** Returns a dispatch function to: in both frontend and backend, add to the current
  * goal's changes the dictionary of words updated with the find-and-replace tool. */
 function addWordChanges(wordChanges: FindAndReplaceChange) {