@zeniai/client-epic-state 5.2.13 → 5.2.14-betaRD0

package/lib/esm/view/expenseAutomationView/epics/jeSchedule/newSchedule/saveNewScheduleEpic.js

@@ -0,0 +1,138 @@
+import { from } from 'rxjs';
+import { catchError, filter, mergeMap, takeUntil } from 'rxjs/operators';
+import { convertToPeriod, toAbsoluteDay, } from '../../../../../commonStateTypes/timePeriod';
+import { getCurrentTenant } from '../../../../../entity/tenant/tenantSelector';
+import { clearAll } from '../../../../../rootActions';
+import { updateJESchedules, updateJESchedulesDetails, } from '../../../../../entity/jeSchedules/jeSchedulesReducer';
+import { getJEScheduleTransactionKey } from '../../../../../entity/jeSchedules/jeSchedulesTypes';
+import { openSnackbar } from '../../../../../entity/snackbar/snackbarReducer';
+import { getScheduleTransactionById } from '../../../../../entity/transaction/transactionSelector';
+import { createZeniAPIStatus, isSuccessResponse, } from '../../../../../responsePayload';
+import { prepareNewSchedulePayload, } from '../../../helpers/newScheduleLocalDataHelper';
+import { fetchJeSchedules, resetNewScheduleLocalData, saveNewSchedule, saveNewScheduleFailure, saveNewScheduleSuccess, } from '../../../reducers/jeSchedulesViewReducer';
+const FAILURE_REASON_MESSAGES = {
+    missing_third_party_id: 'This transaction is missing a QuickBooks reference and cannot be scheduled.',
+    // The form cannot satisfy this: the vendor field is hard-disabled and inherited from the base
+    // transaction, so the fix is on the transaction, not here.
+    missing_vendor: 'This transaction has no vendor and cannot be scheduled. Add a vendor to the transaction first.',
+};
+export const saveNewScheduleEpic = (actions$, state$, zeniAPI) => actions$.pipe(filter(saveNewSchedule.match), mergeMap((action) => {
+    const { scheduleType } = action.payload;
+    const state = state$.value;
+    const localData = state.expenseAutomationJESchedulesViewState.newSchedule
+        .newScheduleState[scheduleType];
+    const { selectedBaseTransactionId } = localData;
+    const transaction = selectedBaseTransactionId != null
+        ? getScheduleTransactionById(state.transactionState, getJEScheduleTransactionKey(selectedBaseTransactionId))
+        : undefined;
+    if (transaction == null) {
+        return from([
+            saveNewScheduleFailure({
+                status: createZeniAPIStatus('Unexpected Error', 'No base transaction selected for this schedule.'),
+            }),
+            openSnackbar({
+                messageSection: 'create_schedule',
+                messageText: 'failed',
+                type: 'error',
+                variables: [
+                    {
+                        variableName: '_api-error_',
+                        variableValue: 'No base transaction selected.',
+                    },
+                ],
+            }),
+        ]);
+    }
+    const result = prepareNewSchedulePayload(transaction, localData, scheduleType);
+    if (!result.ok) {
+        const message = FAILURE_REASON_MESSAGES[result.reason];
+        return from([
+            saveNewScheduleFailure({
+                status: createZeniAPIStatus('Unexpected Error', message),
+            }),
+            openSnackbar({
+                messageSection: 'create_schedule',
+                messageText: 'failed',
+                type: 'error',
+                variables: [{ variableName: '_api-error_', variableValue: message }],
+            }),
+        ]);
+    }
+    const vendorName = localData.selectedVendor?.name ?? transaction.vendorName ?? '';
+    const queryParam = {
+        je_schedule_type: scheduleType,
+    };
+    const payload = { ...result.payload };
+    // convertToPeriod destructures, so an unset period would throw here and take the epic
+    // subscription down for the whole session -- a missed refresh is the cheaper failure.
+    const selectedMonthYear = state.expenseAutomationViewState.selectedPeriodByTenantId[getCurrentTenant(state).tenantId];
+    const selectedPeriod = selectedMonthYear != null
+        ? convertToPeriod(selectedMonthYear)
+        : undefined;
+    const periodStart = selectedPeriod != null
+        ? toAbsoluteDay(selectedPeriod.start)
+        : undefined;
+    const periodEnd = selectedPeriod != null ? toAbsoluteDay(selectedPeriod.end) : undefined;
+    return zeniAPI
+        .postAndGetJSON(`${zeniAPI.apiEndPoints.accountMicroServiceBaseUrl}/1.0/schedules?query=${encodeURIComponent(JSON.stringify(queryParam))}`, payload)
+        .pipe(mergeMap((response) => {
+        if (isSuccessResponse(response) && response.data != null) {
+            const successActions = [
+                updateJESchedules(response.data.je_schedules),
+                updateJESchedulesDetails(response.data.je_schedules, response.data.scheduled_journal_entries),
+                resetNewScheduleLocalData({ scheduleType }),
+                saveNewScheduleSuccess(),
+                openSnackbar({
+                    messageSection: 'create_schedule',
+                    messageText: 'success',
+                    type: 'success',
+                    variables: [
+                        { variableName: '_vendor_name_', variableValue: vendorName },
+                    ],
+                }),
+            ];
+            // Writing the entity is not enough: the list reads
+            // jeScheduleTransactionKeysByPeriodAndTab, which is rebuilt wholesale per fetch, so
+            // without this the new schedule stays invisible until some later fetch. The page
+            // only refetches when the save also changes tab, so saving from the tab the row
+            // lands on showed nothing. Same reasoning as createOneTimeAccrualEpic.
+            if (periodStart != null && periodEnd != null) {
+                successActions.push(fetchJeSchedules({ start: periodStart, end: periodEnd }, true));
+            }
+            return from(successActions);
+        }
+        return from([
+            saveNewScheduleFailure({ status: response.status }),
+            openSnackbar({
+                messageSection: 'create_schedule',
+                messageText: 'failed',
+                type: 'error',
+                variables: [
+                    {
+                        variableName: '_api-error_',
+                        variableValue: response.status.message,
+                    },
+                ],
+            }),
+        ]);
+    }), catchError((error) => from([
+        openSnackbar({
+            messageSection: 'create_schedule',
+            messageText: 'failed',
+            type: 'error',
+            variables: [
+                {
+                    variableName: '_api-error_',
+                    variableValue: JSON.stringify(error),
+                },
+            ],
+        }),
+        saveNewScheduleFailure({
+            status: createZeniAPIStatus('Unexpected Error', `Save New Schedule REST API call errored out ${JSON.stringify(error)}`),
+        }),
+    ])), 
+    // A tenant switch clears the store while this request is still in flight.
+    // Without this the old tenant's response is written back into the freshly
+    // cleared store and stays on screen under the new tenant.
+    takeUntil(actions$.pipe(filter(clearAll.match))));
+}));

package/lib/esm/view/expenseAutomationView/epics/jeSchedule/newSchedule/searchTransactionsForNewScheduleEpic.js

@@ -0,0 +1,79 @@
+import { EMPTY, from, merge, of } from 'rxjs';
+import { catchError, debounceTime, filter, mergeMap, switchMap, } from 'rxjs/operators';
+import { getJEScheduleTransactionKey } from '../../../../../entity/jeSchedules/jeSchedulesTypes';
+import { toTransactionType } from '../../../../../entity/transaction/stateTypes/transactionType';
+import { updateScheduleTransactions } from '../../../../../entity/transaction/transactionReducer';
+import { createZeniAPIStatus, isSuccessResponse, } from '../../../../../responsePayload';
+import { searchTransactionsForNewSchedule, searchTransactionsForNewScheduleFailure, searchTransactionsForNewScheduleSuccess, } from '../../../reducers/jeSchedulesViewReducer';
+import { MIN_NEW_SCHEDULE_SEARCH_LENGTH } from '../../../types/jeSchedulesViewState';
+/**
+ * Dropdown-sized result set. The endpoint is paginated — it takes a `page_token` $skip offset and
+ * returns `next_page_token` — but nothing here reads or sends those, so this is the whole result
+ * set the form ever sees. 20 rather than the backend's default of 10 because a text search for one
+ * known transaction was running out of room; the cap server-side is 100 if this needs raising again
+ * before infinite scroll is wired up.
+ */
+const NEW_SCHEDULE_SEARCH_PAGE_SIZE = 20;
+/** Fallback when the backend sends a null currency for a search result. */
+const DEFAULT_CURRENCY_CODE = 'USD';
+const DEFAULT_CURRENCY_SYMBOL = '$';
+export const searchTransactionsForNewScheduleEpic = (actions$, _state$, zeniAPI) => {
+    const searchActions$ = actions$.pipe(filter(searchTransactionsForNewSchedule.match));
+    /** No API for empty or 1-char queries; immediate merge so switchMap cancels any in-flight HTTP. */
+    const newSearchTooShortOrClear$ = searchActions$.pipe(filter((action) => action.payload.query.trim().length < MIN_NEW_SCHEDULE_SEARCH_LENGTH));
+    const newSearchDebounced$ = searchActions$.pipe(filter((action) => action.payload.query.trim().length >= MIN_NEW_SCHEDULE_SEARCH_LENGTH), debounceTime(300));
+    return merge(newSearchTooShortOrClear$, newSearchDebounced$).pipe(switchMap((action) => {
+        const query = action.payload.query.trim();
+        if (query.length < MIN_NEW_SCHEDULE_SEARCH_LENGTH) {
+            return EMPTY;
+        }
+        const queryParam = {
+            page_size: NEW_SCHEDULE_SEARCH_PAGE_SIZE,
+            search_text: query,
+            je_schedule_type: action.payload.scheduleType,
+        };
+        return zeniAPI
+            .getJSON(`${zeniAPI.apiEndPoints.accountMicroServiceBaseUrl}/1.0/schedules/base_transaction_search?query=${encodeURIComponent(JSON.stringify(queryParam))}`)
+            .pipe(mergeMap((response) => {
+            if (isSuccessResponse(response) && response.data != null) {
+                const { transactions } = response.data;
+                const transactionsPerCurrency = {};
+                transactions.forEach((transaction) => {
+                    const { currency_code, currency_symbol } = transaction;
+                    const currencyCode = currency_code ?? DEFAULT_CURRENCY_CODE;
+                    const existing = transactionsPerCurrency[currencyCode];
+                    transactionsPerCurrency[currencyCode] =
+                        existing != null
+                            ? {
+                                currencySymbol: existing.currencySymbol,
+                                transactions: [...existing.transactions, transaction],
+                            }
+                            : {
+                                currencySymbol: currency_code == null
+                                    ? DEFAULT_CURRENCY_SYMBOL
+                                    : currency_symbol,
+                                transactions: [transaction],
+                            };
+                });
+                const actionsOut = Object.keys(transactionsPerCurrency).map((currencyCode) => {
+                    const perCurrency = transactionsPerCurrency[currencyCode];
+                    return updateScheduleTransactions(perCurrency.transactions, currencyCode, perCurrency.currencySymbol, 'merge');
+                });
+                actionsOut.push(searchTransactionsForNewScheduleSuccess({
+                    searchResults: transactions.map((transaction) => getJEScheduleTransactionKey({
+                        id: transaction.transaction_id,
+                        type: toTransactionType(transaction.transaction_type),
+                        lineId: transaction.line_id,
+                    })),
+                }));
+                return from(actionsOut);
+            }
+            return of(searchTransactionsForNewScheduleFailure({
+                status: response.status,
+            }));
+        }), catchError((error) => of(searchTransactionsForNewScheduleFailure({
+            status: createZeniAPIStatus('Unexpected Error', 'Search transactions for new schedule errored: ' +
+                JSON.stringify(error)),
+        }))));
+    }));
+};

package/lib/esm/view/expenseAutomationView/epics/jeSchedule/oneTimeAccrualActionsEpic.js

@@ -0,0 +1,180 @@
+import { from, merge } from 'rxjs';
+import { catchError, filter, mergeMap, takeUntil } from 'rxjs/operators';
+import { DEFAULT_DATE_FORMAT, convertToPeriod, toAbsoluteDay, } from '../../../../commonStateTypes/timePeriod';
+import { generateJEOneTimeAccrualKey } from '../../../../entity/jeSchedules/jeScheduleHelper';
+import { updateJEOneTimeAccruals } from '../../../../entity/jeSchedules/jeSchedulesReducer';
+import { getJEOneTimeAccrualByKey } from '../../../../entity/jeSchedules/jeSchedulesSelector';
+import { openSnackbar } from '../../../../entity/snackbar/snackbarReducer';
+import { getCurrentTenant } from '../../../../entity/tenant/tenantSelector';
+import { clearAll } from '../../../../rootActions';
+import { createZeniAPIStatus, isSuccessResponse, } from '../../../../responsePayload';
+import { fetchJeSchedules, ignoreOneTimeAccrual, oneTimeAccrualActionFailure, oneTimeAccrualActionSuccess, promoteOneTimeAccrual, revertOneTimeAccrual, } from '../../reducers/jeSchedulesViewReducer';
+/**
+ * The page the accrual came from, so a promoted or ignored row lands in the right bucket. Guarded
+ * because convertToPeriod destructures: an unset period would throw and take the epic subscription
+ * down for the session, and a missed refresh is the cheaper failure.
+ *
+ * Read state$.value at response time rather than the snapshot taken when the action was dispatched:
+ * if the user changes month while the request is in flight, the snapshot refreshes the month they
+ * have left and the month they are now looking at stays stale.
+ */
+function refreshCurrentPage(state) {
+    const selectedMonthYear = state.expenseAutomationViewState.selectedPeriodByTenantId[getCurrentTenant(state).tenantId];
+    if (selectedMonthYear == null) {
+        return [];
+    }
+    const period = convertToPeriod(selectedMonthYear);
+    const start = toAbsoluteDay(period.start);
+    const end = toAbsoluteDay(period.end);
+    if (start == null || end == null) {
+        return [];
+    }
+    return [fetchJeSchedules({ start, end }, true)];
+}
+function failureActions(
+// Per action, not fixed: every failure used to read "Save schedule failed", so skipping or
+// reverting an accrual reported an operation the user had not performed.
+messageSection, accrualId, message, status) {
+    return [
+        oneTimeAccrualActionFailure({
+            accrualId,
+            status: status ?? createZeniAPIStatus('Unexpected Error', message),
+        }),
+        openSnackbar({
+            messageSection,
+            messageText: 'failed',
+            type: 'error',
+            variables: [{ variableName: '_api-error_', variableValue: message }],
+        }),
+    ];
+}
+export const promoteOneTimeAccrualEpic = (actions$, state$, zeniAPI) => actions$.pipe(filter(promoteOneTimeAccrual.match), mergeMap((action) => {
+    const state = state$.value;
+    const { accrualId, amount, edits } = action.payload;
+    const accrual = getJEOneTimeAccrualByKey(generateJEOneTimeAccrualKey(accrualId), state.jeSchedulesState);
+    // An edited value wins; anything untouched falls back to the stored draft, which is what the
+    // list's inline Post sends.
+    const debitAccountId = edits?.jeDebitAccountId ?? accrual?.jeDebitAccountId;
+    const creditAccountId = edits?.jeCreditAccountId ?? accrual?.jeCreditAccountId;
+    if (debitAccountId == null || creditAccountId == null) {
+        // Promote posts a journal entry, so it cannot proceed without both sides of it.
+        return from(failureActions('save_schedule', accrualId, 'This accrual is missing an expense or liability account.'));
+    }
+    // vendor_id, vendor_name and service_month are deliberately absent: promote takes them from
+    // the stored draft and ignores them in the body, so sending them would imply a reassignment
+    // that never happens.
+    // Untouched falls back to the stored policy, so posting without editing cannot silently
+    // flip a scheduled reversal to on-bill-match.
+    const reverseNextMonth = edits?.reverseNextMonth ??
+        accrual?.reversalPolicy === 'scheduled_date';
+    const expectedReversalDate = edits?.expectedReversalDate ?? accrual?.expectedReversalDate;
+    const debitClassId = edits?.jeDebitAccountingClassId ?? accrual?.jeDebitAccountingClassId;
+    const debitClassName = edits?.jeDebitAccountingClassName ??
+        accrual?.jeDebitAccountingClassName;
+    const payload = {
+        amount,
+        memo: edits?.memo ?? accrual?.memo ?? '',
+        je_debit_account_id: debitAccountId,
+        je_credit_account_id: creditAccountId,
+        ...(debitClassId != null ? { je_debit_accounting_class_id: debitClassId } : {}),
+        ...(debitClassName != null
+            ? { je_debit_accounting_class_name: debitClassName }
+            : {}),
+        // Mirrors the create payload: reverse_next_month alone decides the policy, and the date is
+        // only meaningful when it is true -- agent-je derives the reversal from the bill otherwise.
+        reverse_next_month: reverseNextMonth,
+        ...(reverseNextMonth && expectedReversalDate != null
+            ? {
+                expected_reversal_date: expectedReversalDate.format(DEFAULT_DATE_FORMAT),
+            }
+            : {}),
+    };
+    return zeniAPI
+        .putAndGetJSON(`${zeniAPI.apiEndPoints.accountMicroServiceBaseUrl}/1.0/accruals/${accrualId}`, { ...payload })
+        .pipe(mergeMap((response) => {
+        if (isSuccessResponse(response) && response.data != null) {
+            return from([
+                updateJEOneTimeAccruals([response.data.accrual]),
+                oneTimeAccrualActionSuccess({ accrualId }),
+                openSnackbar({
+                    messageSection: 'save_schedule',
+                    messageText: 'success',
+                    type: 'success',
+                    variables: [
+                        {
+                            variableName: '_vendor_name_',
+                            variableValue: accrual?.vendorName ?? '',
+                        },
+                    ],
+                }),
+                ...refreshCurrentPage(state$.value),
+            ]);
+        }
+        return from(failureActions('save_schedule', accrualId, response.status.message, response.status));
+    }), catchError((error) => from(failureActions('save_schedule', accrualId, `Promote accrual REST API call errored out ${JSON.stringify(error)}`))), 
+    // A tenant switch clears the store while this request is still in flight.
+    // Without this the old tenant's response is written back into the freshly
+    // cleared store and stays on screen under the new tenant.
+    takeUntil(actions$.pipe(filter(clearAll.match))));
+}));
+export const ignoreOneTimeAccrualEpic = (actions$, state$, zeniAPI) => actions$.pipe(filter(ignoreOneTimeAccrual.match), mergeMap((action) => {
+    const { accrualId, reason, note } = action.payload;
+    // `scope` is deliberately never sent: the API returns 400 on it, which is why the design's
+    // "Apply to" selector cannot be built yet.
+    const payload = {
+        reason,
+        ...(note != null && note !== '' ? { note } : {}),
+    };
+    return zeniAPI
+        .putAndGetJSON(`${zeniAPI.apiEndPoints.accountMicroServiceBaseUrl}/1.0/accruals/${accrualId}/ignore`, { ...payload })
+        .pipe(mergeMap((response) => {
+        if (isSuccessResponse(response) && response.data != null) {
+            return from([
+                updateJEOneTimeAccruals([response.data.accrual]),
+                oneTimeAccrualActionSuccess({ accrualId }),
+                openSnackbar({
+                    messageSection: 'ignore_accrual',
+                    messageText: 'success',
+                    type: 'success',
+                }),
+                ...refreshCurrentPage(state$.value),
+            ]);
+        }
+        return from(failureActions('ignore_accrual', accrualId, response.status.message, response.status));
+    }), catchError((error) => from(failureActions('ignore_accrual', accrualId, `Ignore accrual REST API call errored out ${JSON.stringify(error)}`))), 
+    // A tenant switch clears the store while this request is still in flight.
+    // Without this the old tenant's response is written back into the freshly
+    // cleared store and stays on screen under the new tenant.
+    takeUntil(actions$.pipe(filter(clearAll.match))));
+}));
+export const revertOneTimeAccrualEpic = (actions$, state$, zeniAPI) => actions$.pipe(filter(revertOneTimeAccrual.match), mergeMap((action) => {
+    const { accrualId, reason, note } = action.payload;
+    // Deletes the accrual's journal entry in QBO and soft-deletes the row. `scope` is never sent:
+    // the endpoint 400s on it, exactly as ignore does.
+    const payload = {
+        reason,
+        ...(note != null && note !== '' ? { note } : {}),
+    };
+    return zeniAPI
+        .putAndGetJSON(`${zeniAPI.apiEndPoints.accountMicroServiceBaseUrl}/1.0/accruals/${accrualId}/revert`, { ...payload })
+        .pipe(mergeMap((response) => {
+        if (isSuccessResponse(response) && response.data != null) {
+            return from([
+                updateJEOneTimeAccruals([response.data.accrual]),
+                oneTimeAccrualActionSuccess({ accrualId }),
+                openSnackbar({
+                    messageSection: 'revert_accrual',
+                    messageText: 'success',
+                    type: 'success',
+                }),
+                ...refreshCurrentPage(state$.value),
+            ]);
+        }
+        return from(failureActions('revert_accrual', accrualId, response.status.message, response.status));
+    }), catchError((error) => from(failureActions('revert_accrual', accrualId, `Revert accrual REST API call errored out ${JSON.stringify(error)}`))), 
+    // A tenant switch clears the store while this request is still in flight.
+    // Without this the old tenant's response is written back into the freshly
+    // cleared store and stays on screen under the new tenant.
+    takeUntil(actions$.pipe(filter(clearAll.match))));
+}));
+export const oneTimeAccrualActionsEpic = (actions$, state$, zeniAPI) => merge(promoteOneTimeAccrualEpic(actions$, state$, zeniAPI), ignoreOneTimeAccrualEpic(actions$, state$, zeniAPI), revertOneTimeAccrualEpic(actions$, state$, zeniAPI));

package/lib/esm/view/expenseAutomationView/epics/jeSchedule/revertJeScheduleEpic.js

@@ -0,0 +1,111 @@
+import { from } from 'rxjs';
+import { catchError, filter, mergeMap, takeUntil } from 'rxjs/operators';
+import { openSnackbar } from '../../../../entity/snackbar/snackbarReducer';
+import { getCurrentTenant } from '../../../../entity/tenant/tenantSelector';
+import { clearAll } from '../../../../rootActions';
+import { createZeniAPIStatus, isSuccessResponse, } from '../../../../responsePayload';
+import { fetchMonthEndCloseChecks } from '../../../monthEndCloseChecksView/monthEndCloseChecksViewReducer';
+import { removeJeAccruedScheduleKey, removeJeScheduleTransactionKey, revertJeSchedule, revertJeScheduleFailure, revertJeScheduleSuccess, } from '../../reducers/jeSchedulesViewReducer';
+/**
+ * Undo a posted prepaid / fixed-asset / accrued schedule: deletes its posted QBO journal entry, then
+ * soft-deletes the scheduled entries and the schedule itself.
+ *
+ * Two or more posted journal entries answer 409 and leave the schedule intact, so the failure is
+ * surfaced rather than swallowed. Accrued has one extra rejection of its own — the endpoint refuses
+ * once a bill has been linked, because the reversal is already on the books — and that arrives the
+ * same way, as an error on the row.
+ *
+ * A one-time accrual is not this action: it is a Mongo row behind the accruals gateway, and goes
+ * through revertOneTimeAccrual instead. So `accrued_expenses` here always means an accrued
+ * *schedule*, which is what makes it a safe discriminator for the removal below.
+ */
+export const revertJeScheduleEpic = (actions$, state$, zeniAPI) => actions$.pipe(filter(revertJeSchedule.match), mergeMap((action) => {
+    const state = state$.value;
+    const { expenseAutomationViewState: { selectedPeriodByTenantId }, } = state;
+    const { jeScheduleId, jeScheduleTransactionKey, reason, note, scheduleType } = action.payload;
+    const currentTenant = getCurrentTenant(state);
+    const selectedPeriod = selectedPeriodByTenantId[currentTenant.tenantId];
+    // Feedback nested under je_schedules[0], the same shape ignore takes -- the two endpoints were
+    // made symmetric server-side. A flat {reason, note} body, which this epic used to send while
+    // revert validated the whole body as feedback, now answers 400 "Request body must contain at
+    // least one of je_schedules".
+    //
+    // Only the id and the feedback: the rest of the fat blob is read from Datastore, not the
+    // request. The id is optional server-side but sent anyway, because it is validated against the
+    // URL and so turns a mismatch into a 400 rather than a revert of the wrong schedule.
+    // `action` is stamped server-side.
+    const payload = {
+        je_schedules: [
+            {
+                je_schedule_id: jeScheduleId,
+                feedback: {
+                    reason,
+                    ...(note != null && note !== '' ? { note } : {}),
+                },
+            },
+        ],
+    };
+    return zeniAPI
+        .putAndGetJSON(`${zeniAPI.apiEndPoints.accountMicroServiceBaseUrl}/1.0/schedules/${jeScheduleId}/revert`, payload)
+        .pipe(mergeMap((response) => {
+        if (isSuccessResponse(response) && response.data != null) {
+            // Deliberately not written to the entity store. A reverted schedule is soft-deleted,
+            // so the endpoint returns its flat entity json -- `vendor_id` as a bare field, no
+            // nested `vendor` or `base_transaction` -- rather than the list DTO those mappers
+            // expect. Feeding it to updateJESchedules threw inside doUpdateVendors on
+            // `payload.vendor.vendor_id`, and because that happens mid-dispatch it killed the
+            // rest of this array: the row kept spinning and no snackbar appeared. The row is
+            // removed from the list below regardless, so there is nothing worth storing.
+            return from([
+                revertJeScheduleSuccess({ jeScheduleTransactionKey }),
+                // Accrued keys live in their own per-tab map, so the transaction-key removal cannot
+                // reach them -- a reverted accrued schedule would sit on the list until the next
+                // fetch. The key carried here is a JEScheduleKey for that type.
+                scheduleType === 'accrued_expenses'
+                    ? removeJeAccruedScheduleKey(jeScheduleTransactionKey)
+                    : removeJeScheduleTransactionKey(jeScheduleTransactionKey, selectedPeriod),
+                fetchMonthEndCloseChecks({
+                    tenantId: currentTenant.tenantId,
+                    period: selectedPeriod,
+                    refreshViewInBackground: true,
+                }),
+                openSnackbar({
+                    messageSection: 'revert_schedule',
+                    messageText: 'success',
+                    type: 'success',
+                }),
+            ]);
+        }
+        return from([
+            revertJeScheduleFailure({
+                jeScheduleTransactionKey,
+                status: response.status,
+            }),
+            openSnackbar({
+                messageSection: 'revert_schedule',
+                messageText: 'failed',
+                type: 'error',
+                variables: [
+                    {
+                        variableName: '_api-error_',
+                        variableValue: response.status.message,
+                    },
+                ],
+            }),
+        ]);
+    }), catchError((error) => from([
+        revertJeScheduleFailure({
+            jeScheduleTransactionKey,
+            status: createZeniAPIStatus('Unexpected Error', `Revert schedule REST API call errored out ${JSON.stringify(error)}`),
+        }),
+        openSnackbar({
+            messageSection: 'revert_schedule',
+            messageText: 'failed',
+            type: 'error',
+        }),
+    ])), 
+    // A tenant switch clears the store while this request is still in flight.
+    // Without this the old tenant's response is written back into the freshly
+    // cleared store and stays on screen under the new tenant.
+    takeUntil(actions$.pipe(filter(clearAll.match))));
+}));

package/lib/esm/view/expenseAutomationView/epics/jeSchedule/switchJESchedulesTabEpic.js

@@ -0,0 +1,28 @@
+import { EMPTY, of } from 'rxjs';
+import { filter, mergeMap } from 'rxjs/operators';
+import { convertToPeriod, toAbsoluteDay, } from '../../../../commonStateTypes/timePeriod';
+import { getCurrentTenant } from '../../../../entity/tenant/tenantSelector';
+import { fetchJeSchedulesPage, updateJESchedulesUIState, } from '../../reducers/jeSchedulesViewReducer';
+export const switchJESchedulesTabEpic = (actions$, state$) => actions$.pipe(filter(updateJESchedulesUIState.match), filter((action) => action.payload.selectedPageTab != null), mergeMap(() => {
+    const state = state$.value;
+    const { fetchState, refreshStatus } = state.expenseAutomationJESchedulesViewState;
+    if (fetchState === 'In-Progress' ||
+        refreshStatus.fetchState === 'In-Progress') {
+        return EMPTY;
+    }
+    const currentTenant = getCurrentTenant(state);
+    const selectedPeriod = state.expenseAutomationViewState.selectedPeriodByTenantId[currentTenant.tenantId];
+    // convertToPeriod destructures, so an unset period throws and takes this epic's subscription
+    // down for the rest of the session -- every later tab switch would then stop refreshing. A
+    // skipped refresh is the cheaper failure, as in refreshCurrentPage.
+    if (selectedPeriod == null) {
+        return EMPTY;
+    }
+    const period = convertToPeriod(selectedPeriod);
+    const timePeriodStart = toAbsoluteDay(period.start);
+    const timePeriodEnd = toAbsoluteDay(period.end);
+    if (timePeriodStart == null || timePeriodEnd == null) {
+        return EMPTY;
+    }
+    return of(fetchJeSchedulesPage({ start: timePeriodStart, end: timePeriodEnd }, true));
+}));