@zeniai/client-epic-state 4.19.84-betaVR7 → 4.19.84-betaVR9

package/lib/esm/view/expenseAutomationView/epics/missingReceipts/bulkUploadReceiptsEpic.js

@@ -12,9 +12,13 @@ export const bulkUploadReceiptsEpic = (actions$, _state$, zeniAPI) => actions$.p
         .postFormData(`${zeniAPI.apiEndPoints.communicationAgentMicroServiceBaseUrl}/1.0/receipts/bulk-upload`, formData)
         .pipe(mergeMap((response) => {
         if (isSuccessResponse(response) && response.data != null) {
-            return of(bulkUploadReceiptsSuccess({
-                batchId: response.data.batch_id,
-            }));
+            const batchId = String(response.data.batch_id ?? '').trim();
+            if (batchId === '') {
+                return of(bulkUploadReceiptsFailure({
+                    error: createZeniAPIStatus('Unexpected Error', 'Bulk upload succeeded but batch_id was missing or empty.'),
+                }));
+            }
+            return of(bulkUploadReceiptsSuccess({ batchId }));
         }
         return of(bulkUploadReceiptsFailure({ error: response.status }));
     }), catchError((error) => of(bulkUploadReceiptsFailure({

package/lib/esm/view/expenseAutomationView/epics/missingReceipts/watchBulkUploadBatchStatusEpic.js

@@ -1,5 +1,6 @@
 import { EMPTY, from, merge, of, timer } from 'rxjs';
 import { catchError, debounceTime, filter, mergeMap, switchMap, takeUntil, } from 'rxjs/operators';
+import { openSnackbar } from '../../../../entity/snackbar/snackbarReducer';
 import { updateTransactions } from '../../../../entity/transaction/transactionReducer';
 import { isSuccessResponse } from '../../../../responsePayload';
 import { extractTransactionPayloadsFromBatchFiles, isBatchDetailsApiStatusCompleted, toBatchDetails, } from '../../payload/missingReceiptsPayload';
@@ -18,8 +19,14 @@ export const pusherBatchStatusCompletionEpic = (actions$) => actions$.pipe(filte
     }),
     requestMissingReceiptsTabNavigation({ tab: 'unmatched' }),
 ])));
+/**
+ * Fallback polling when Pusher is slow or unavailable. Network errors on a poll tick show one
+ * error snackbar per upload session (first failure only) so the user is not spammed every
+ * interval; Pusher or a later successful poll still completes the flow.
+ */
 export const pollBulkUploadBatchStatusEpic = (actions$, _state$, zeniAPI) => actions$.pipe(filter(bulkUploadReceiptsSuccess.match), mergeMap((action) => {
     const { batchId } = action.payload;
+    let pollErrorNotified = false;
     return timer(FALLBACK_FIRST_DELAY_MS, FALLBACK_POLL_INTERVAL_MS).pipe(takeUntil(merge(actions$.pipe(filter(clearBulkUpload.match)), actions$.pipe(filter(pusherBatchStatusUpdate.match), filter((a) => a.payload.batchId === batchId)), actions$.pipe(filter(fetchBulkUploadBatchDetailsSuccess.match), filter((a) => a.payload.batchId === batchId)))), switchMap(() => zeniAPI
         .getJSON(`${zeniAPI.apiEndPoints.communicationAgentMicroServiceBaseUrl}/1.0/batches/${batchId}`)
         .pipe(mergeMap((response) => {
@@ -38,5 +45,15 @@ export const pollBulkUploadBatchStatusEpic = (actions$, _state$, zeniAPI) => act
             }
         }
         return EMPTY;
-    }), catchError(() => of()))));
+    }), catchError(() => {
+        if (!pollErrorNotified) {
+            pollErrorNotified = true;
+            return of(openSnackbar({
+                messageSection: 'receipts_upload',
+                messageText: 'failed',
+                type: 'error',
+            }));
+        }
+        return EMPTY;
+    }))));
 }));

package/lib/esm/view/expenseAutomationView/payload/missingReceiptsPayload.js

@@ -1,6 +1,36 @@
 import { toURL } from '../../../commonPayloadTypes/urlPayload';
 import { getTransactionPayloadAmount, } from '../../../entity/transaction/payloadTypes/transactionPayload';
 // -- Converter functions --
+/**
+ * Maps batch-details API `files[].status` strings to canonical {@link BatchFile} statuses.
+ * The service may send aliases (`unmatched`, `possible_match`) or different separators; the UI
+ * only treats `un_matched` / `no_match` as work for the Unmatched tab.
+ *
+ * **Unknown values:** Any string not matched below is returned as-is (typed as
+ * {@link BatchFile} status). Callers should not assume exhaustiveness—treat unknown statuses in
+ * UI as “other” (e.g. show a generic label or fall back to a safe bucket) until the API contract
+ * adds an explicit mapping.
+ */
+export function normalizeBatchFileStatus(raw) {
+    const s = raw.trim().toLowerCase().replace(/-/g, '_');
+    switch (s) {
+        case 'un_matched':
+        case 'unmatched':
+            return 'un_matched';
+        case 'no_match':
+        case 'nomatch':
+            return 'no_match';
+        case 'matched':
+            return 'matched';
+        case 'failed':
+            return 'failed';
+        case 'possible_match':
+        case 'possiblematch':
+            return 'un_matched';
+        default:
+            return raw;
+    }
+}
 export function toBatchFile(payload, filesEndPoint) {
     return {
         attachmentId: payload.attachment_id,
@@ -16,7 +46,7 @@ export function toBatchFile(payload, filesEndPoint) {
         filename: payload.filename,
         matchSource: payload.match_source,
         matchedTransactionId: payload.matched_transaction?.transaction_id,
-        status: payload.status,
+        status: normalizeBatchFileStatus(payload.status),
     };
 }
 export function extractTransactionPayloadsFromBatchFiles(files) {