@zeniai/client-epic-state 5.1.92-betaRD10 → 5.1.92-betaRD12

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

@@ -15,7 +15,11 @@ export const initializeNewScheduleEpic = (actions$, state$) => actions$.pipe(fil
     if (classListState.fetchState === 'Not-Started') {
         newScheduleActions.push(fetchClassList());
     }
-    newScheduleActions.push(resetNewScheduleLocalData({ scheduleType: action.payload.scheduleType }));
-    newScheduleActions.push(clearSearchTransactionsForNewSchedule());
+    // The account and class fetches above are idempotent (both guard on 'Not-Started'), so they
+    // can run on every open. Clearing what the user entered cannot, so it is opt-in.
+    if (action.payload.shouldResetEnteredData) {
+        newScheduleActions.push(resetNewScheduleLocalData({ scheduleType: action.payload.scheduleType }));
+        newScheduleActions.push(clearSearchTransactionsForNewSchedule());
+    }
     return from(newScheduleActions);
 }));

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

@@ -6,8 +6,14 @@ import { updateScheduleTransactions } from '../../../../../entity/transaction/tr
 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 — no pagination for Base Transaction search. */
-const NEW_SCHEDULE_SEARCH_PAGE_SIZE = 10;
+/**
+ * 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 = '$';

package/lib/esm/view/expenseAutomationView/reducers/jeSchedulesViewReducer.js

@@ -460,12 +460,12 @@ const expenseAutomationJESchedulesView = createSlice({
             draft.accountSettings.localData = accountSettingsInitialState.localData;
         },
         initializeNewSchedule: {
-            prepare({ scheduleType, }) {
-                return { payload: { scheduleType } };
+            prepare({ scheduleType, shouldResetEnteredData = true, }) {
+                return { payload: { scheduleType, shouldResetEnteredData } };
             },
             reducer() {
                 // Purely an epic-trigger action — the epic fans out the account/class
-                // list fetches plus resetNewScheduleLocalData.
+                // list fetches, and clears entered data only when asked to.
             },
         },
         resetNewScheduleLocalData: {

package/lib/esm/view/expenseAutomationView/selectors/jeSchedulesViewSelector.js

@@ -194,7 +194,7 @@ export function getExpenseAutomationJESchedulesView(state) {
     const accruedSchedules = [];
     const oneTimeAccruals = [];
     const { accountState, accountListState, classState, classListState, expenseAutomationJESchedulesViewState, expenseAutomationViewState, monthEndCloseChecksState, monthEndCloseChecksViewState, } = state;
-    const { jeScheduleTransactionKeysByPeriodAndTab, jeScheduleAccruedKeysByPeriodAndTab, jeScheduleOneTimeAccrualKeysByPeriodAndTab, failedJeScheduleTransactionKeysByPeriodAndTab, jeScheduleLocalDataById, postStatusById, ignoreStatusById, revertStatusById, uiState, error, fetchState, refreshStatus, newSchedule, } = expenseAutomationJESchedulesViewState;
+    const { jeScheduleTransactionKeysByPeriodAndTab, jeScheduleAccruedKeysByPeriodAndTab, jeScheduleOneTimeAccrualKeysByPeriodAndTab, failedJeScheduleTransactionKeysByPeriodAndTab, jeScheduleLocalDataById, postStatusById, ignoreStatusById, revertStatusById, oneTimeAccrualActionStatusById, uiState, error, fetchState, refreshStatus, newSchedule, } = expenseAutomationJESchedulesViewState;
     const accountSettingsView = getJEAccountSettingsView(state);
     const newScheduleView = newSchedule.selectedDetailScheduleType === 'fixed_assets' ||
         newSchedule.selectedDetailScheduleType === 'prepaid_expenses'
@@ -409,6 +409,7 @@ export function getExpenseAutomationJESchedulesView(state) {
         postStatusById: postStatusById,
         ignoreStatusById: ignoreStatusById,
         revertStatusById: revertStatusById,
+        oneTimeAccrualActionStatusById: oneTimeAccrualActionStatusById,
         fetchState: reducedFetchStatus.fetchState,
         jeScheduleLocalDataById: jeScheduleLocalDataById,
         uiState: uiState,

package/lib/view/expenseAutomationView/epics/jeSchedule/newSchedule/initializeNewScheduleEpic.js

@@ -18,8 +18,12 @@ const initializeNewScheduleEpic = (actions$, state$) => actions$.pipe((0, operat
     if (classListState.fetchState === 'Not-Started') {
         newScheduleActions.push((0, classListReducer_1.fetchClassList)());
     }
-    newScheduleActions.push((0, jeSchedulesViewReducer_1.resetNewScheduleLocalData)({ scheduleType: action.payload.scheduleType }));
-    newScheduleActions.push((0, jeSchedulesViewReducer_1.clearSearchTransactionsForNewSchedule)());
+    // The account and class fetches above are idempotent (both guard on 'Not-Started'), so they
+    // can run on every open. Clearing what the user entered cannot, so it is opt-in.
+    if (action.payload.shouldResetEnteredData) {
+        newScheduleActions.push((0, jeSchedulesViewReducer_1.resetNewScheduleLocalData)({ scheduleType: action.payload.scheduleType }));
+        newScheduleActions.push((0, jeSchedulesViewReducer_1.clearSearchTransactionsForNewSchedule)());
+    }
     return (0, rxjs_1.from)(newScheduleActions);
 }));
 exports.initializeNewScheduleEpic = initializeNewScheduleEpic;

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

@@ -9,8 +9,14 @@ const transactionReducer_1 = require("../../../../../entity/transaction/transact
 const responsePayload_1 = require("../../../../../responsePayload");
 const jeSchedulesViewReducer_1 = require("../../../reducers/jeSchedulesViewReducer");
 const jeSchedulesViewState_1 = require("../../../types/jeSchedulesViewState");
-/** Dropdown-sized result set — no pagination for Base Transaction search. */
-const NEW_SCHEDULE_SEARCH_PAGE_SIZE = 10;
+/**
+ * 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 = '$';

package/lib/view/expenseAutomationView/reducers/jeSchedulesViewReducer.d.ts

@@ -81,8 +81,10 @@ export declare const clearExpenseAutomationJESchedulesView: import("@reduxjs/too
     jeScheduleTransactionKeys: (`${string}_expense_${string}` | `${string}_check_${string}` | `${string}_refund_${string}` | `${string}_invoice_${string}` | `${string}_payment_${string}` | `${string}_credit_memo_${string}` | `${string}_sales_receipt_${string}` | `${string}_delayed_credit_${string}` | `${string}_delayed_charge_${string}` | `${string}_bill_${string}` | `${string}_bill_payment_${string}` | `${string}_bill_payment_check_${string}` | `${string}_bill_payment_credit_card_${string}` | `${string}_vendor_credit_${string}` | `${string}_credit_card_credit_${string}` | `${string}_transfer_${string}` | `${string}_credit_card_payment_${string}` | `${string}_deposit_${string}` | `${string}_journal_entry_${string}`)[];
 }, "expenseAutomationJESchedulesView/initializeJeScheduleLocalData", never, never>, initializeNewSchedule: import("@reduxjs/toolkit").ActionCreatorWithPreparedPayload<[{
     scheduleType: "fixed_assets" | "prepaid_expenses";
+    shouldResetEnteredData?: boolean;
 }], {
     scheduleType: "prepaid_expenses" | "fixed_assets";
+    shouldResetEnteredData: boolean;
 }, "expenseAutomationJESchedulesView/initializeNewSchedule", never, never>, removeFailedJeScheduleTransactionKey: import("@reduxjs/toolkit").ActionCreatorWithPreparedPayload<[failedJeScheduleKey: `${string}_expense_${string}` | `${string}_check_${string}` | `${string}_refund_${string}` | `${string}_invoice_${string}` | `${string}_payment_${string}` | `${string}_credit_memo_${string}` | `${string}_sales_receipt_${string}` | `${string}_delayed_credit_${string}` | `${string}_delayed_charge_${string}` | `${string}_bill_${string}` | `${string}_bill_payment_${string}` | `${string}_bill_payment_check_${string}` | `${string}_bill_payment_credit_card_${string}` | `${string}_vendor_credit_${string}` | `${string}_credit_card_credit_${string}` | `${string}_transfer_${string}` | `${string}_credit_card_payment_${string}` | `${string}_deposit_${string}` | `${string}_journal_entry_${string}`, selectedPeriod: MonthYearPeriod], {
     failedJeScheduleKey: `${string}_expense_${string}` | `${string}_check_${string}` | `${string}_refund_${string}` | `${string}_invoice_${string}` | `${string}_payment_${string}` | `${string}_credit_memo_${string}` | `${string}_sales_receipt_${string}` | `${string}_delayed_credit_${string}` | `${string}_delayed_charge_${string}` | `${string}_bill_${string}` | `${string}_bill_payment_${string}` | `${string}_bill_payment_check_${string}` | `${string}_bill_payment_credit_card_${string}` | `${string}_vendor_credit_${string}` | `${string}_credit_card_credit_${string}` | `${string}_transfer_${string}` | `${string}_credit_card_payment_${string}` | `${string}_deposit_${string}` | `${string}_journal_entry_${string}`;
     selectedPeriod: MonthYearPeriod;

package/lib/view/expenseAutomationView/reducers/jeSchedulesViewReducer.js

@@ -466,12 +466,12 @@ const expenseAutomationJESchedulesView = (0, toolkit_1.createSlice)({
             draft.accountSettings.localData = exports.accountSettingsInitialState.localData;
         },
         initializeNewSchedule: {
-            prepare({ scheduleType, }) {
-                return { payload: { scheduleType } };
+            prepare({ scheduleType, shouldResetEnteredData = true, }) {
+                return { payload: { scheduleType, shouldResetEnteredData } };
             },
             reducer() {
                 // Purely an epic-trigger action — the epic fans out the account/class
-                // list fetches plus resetNewScheduleLocalData.
+                // list fetches, and clears entered data only when asked to.
             },
         },
         resetNewScheduleLocalData: {

package/lib/view/expenseAutomationView/selectorTypes/jeSchedulesSelectorTypes.d.ts

@@ -48,6 +48,7 @@ export interface ExpenseAutomationJESchedulesViewSelector extends SelectorView {
     };
     kpiSummary: JEKpiSummary;
     newScheduleView: NewScheduleView | undefined;
+    oneTimeAccrualActionStatusById: Record<ID, FetchStateAndError>;
     pendingReviewSchedules: JEScheduleRow[];
     pendingReviewTotal: number;
     postStatusById: Record<ID, FetchStateAndError>;

package/lib/view/expenseAutomationView/selectors/jeSchedulesViewSelector.js

@@ -209,7 +209,7 @@ function getExpenseAutomationJESchedulesView(state) {
     const accruedSchedules = [];
     const oneTimeAccruals = [];
     const { accountState, accountListState, classState, classListState, expenseAutomationJESchedulesViewState, expenseAutomationViewState, monthEndCloseChecksState, monthEndCloseChecksViewState, } = state;
-    const { jeScheduleTransactionKeysByPeriodAndTab, jeScheduleAccruedKeysByPeriodAndTab, jeScheduleOneTimeAccrualKeysByPeriodAndTab, failedJeScheduleTransactionKeysByPeriodAndTab, jeScheduleLocalDataById, postStatusById, ignoreStatusById, revertStatusById, uiState, error, fetchState, refreshStatus, newSchedule, } = expenseAutomationJESchedulesViewState;
+    const { jeScheduleTransactionKeysByPeriodAndTab, jeScheduleAccruedKeysByPeriodAndTab, jeScheduleOneTimeAccrualKeysByPeriodAndTab, failedJeScheduleTransactionKeysByPeriodAndTab, jeScheduleLocalDataById, postStatusById, ignoreStatusById, revertStatusById, oneTimeAccrualActionStatusById, uiState, error, fetchState, refreshStatus, newSchedule, } = expenseAutomationJESchedulesViewState;
     const accountSettingsView = (0, jeAccountSettingsViewSelector_1.getJEAccountSettingsView)(state);
     const newScheduleView = newSchedule.selectedDetailScheduleType === 'fixed_assets' ||
         newSchedule.selectedDetailScheduleType === 'prepaid_expenses'
@@ -424,6 +424,7 @@ function getExpenseAutomationJESchedulesView(state) {
         postStatusById: postStatusById,
         ignoreStatusById: ignoreStatusById,
         revertStatusById: revertStatusById,
+        oneTimeAccrualActionStatusById: oneTimeAccrualActionStatusById,
         fetchState: reducedFetchStatus.fetchState,
         jeScheduleLocalDataById: jeScheduleLocalDataById,
         uiState: uiState,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zeniai/client-epic-state",
-  "version": "5.1.92-betaRD10",
+  "version": "5.1.92-betaRD12",
   "description": "Shared module between Web & Mobile containing required abstractions for state management, async network communication. ",
   "main": "lib/index.js",
   "module": "lib/esm/index.js",