@zeniai/client-epic-state 5.3.8-beta0DI → 5.3.9

package/lib/init.d.ts

@@ -3,6 +3,8 @@ import { RootState } from './rootStateTypes';
 import { RESTAPIEndpoints } from './zeniAPI';
 export type ZeniStore = Store<RootState>;
 export { zeniAPI } from './zeniAPIClientState';
+export { setSessionEndpointsLoader } from './sessionEndpointsLoader';
+export type { SessionEndpointsLoader } from './sessionEndpointsLoader';
 export { disconnectPusher, initializePusher, pusher } from './pusherClient';
 export type { PusherAuthPayload } from './pusherClient';
 export declare let store: ZeniStore;
@@ -23,6 +25,13 @@ export { injectFeatureModules, injectFeatureEpics } from './configureStore';
  * @param tenantId
  */
 export declare function updateZeniAPIClientConfig(userId?: string, sessionId?: string, tenantId?: string): void;
+/**
+ * Replace the REST endpoints the API client calls. Epics read the endpoints
+ * at request time, so every request made after this uses the new ones.
+ * Note: This version SHOULD BE used only in the client side.
+ * @param restEndPoints
+ */
+export declare function updateZeniAPIEndpoints(restEndPoints: RESTAPIEndpoints): void;
 /**
  * Returns current Zeni REST API client config.
  * @returns

package/lib/init.js

@@ -36,9 +36,10 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.injectFeatureEpics = exports.injectFeatureModules = exports.store = exports.pusher = exports.initializePusher = exports.disconnectPusher = exports.zeniAPI = void 0;
+exports.injectFeatureEpics = exports.injectFeatureModules = exports.store = exports.pusher = exports.initializePusher = exports.disconnectPusher = exports.setSessionEndpointsLoader = exports.zeniAPI = void 0;
 exports.default = initialize;
 exports.updateZeniAPIClientConfig = updateZeniAPIClientConfig;
+exports.updateZeniAPIEndpoints = updateZeniAPIEndpoints;
 exports.zeniAPIClientConfig = zeniAPIClientConfig;
 exports.initializeWithNewStore = initializeWithNewStore;
 exports.initialize3rdPartyExtensions = initialize3rdPartyExtensions;
@@ -47,6 +48,8 @@ const zeniAPI_1 = require("./zeniAPI");
 const zeniAPIClientState_1 = require("./zeniAPIClientState");
 var zeniAPIClientState_2 = require("./zeniAPIClientState");
 Object.defineProperty(exports, "zeniAPI", { enumerable: true, get: function () { return zeniAPIClientState_2.zeniAPI; } });
+var sessionEndpointsLoader_1 = require("./sessionEndpointsLoader");
+Object.defineProperty(exports, "setSessionEndpointsLoader", { enumerable: true, get: function () { return sessionEndpointsLoader_1.setSessionEndpointsLoader; } });
 var pusherClient_1 = require("./pusherClient");
 Object.defineProperty(exports, "disconnectPusher", { enumerable: true, get: function () { return pusherClient_1.disconnectPusher; } });
 Object.defineProperty(exports, "initializePusher", { enumerable: true, get: function () { return pusherClient_1.initializePusher; } });
@@ -60,11 +63,10 @@ let initialized = false;
  */
 function initialize(restEndPoints, userId, sessionId, tenantId, isSandboxEnv, onSessionInvalidated) {
     if (initialized) {
-        console.warn(`Already initialized with url ${JSON.stringify(zeniAPIClientState_1.zeniAPI.apiEndPoints)}`);
+        console.warn('client-epic-state is already initialized');
         return exports.store;
     }
     initialized = true;
-    console.info(`initializing with ${JSON.stringify(restEndPoints)}`);
     initialize3rdPartyExtensions();
     (0, zeniAPIClientState_1.setZeniAPIClient)(new zeniAPI_1.ZeniAPIClient(restEndPoints, userId, sessionId, tenantId, onSessionInvalidated));
     exports.store = (0, configureStore_1.default)(zeniAPIClientState_1.zeniAPI, isSandboxEnv);
@@ -84,6 +86,18 @@ Object.defineProperty(exports, "injectFeatureEpics", { enumerable: true, get: fu
 function updateZeniAPIClientConfig(userId, sessionId, tenantId) {
     zeniAPIClientState_1.zeniAPI.update(userId, sessionId, tenantId);
 }
+/**
+ * Replace the REST endpoints the API client calls. Epics read the endpoints
+ * at request time, so every request made after this uses the new ones.
+ * Note: This version SHOULD BE used only in the client side.
+ * @param restEndPoints
+ */
+function updateZeniAPIEndpoints(restEndPoints) {
+    if (zeniAPIClientState_1.zeniAPI == null) {
+        throw new Error('Call initialize() before updateZeniAPIEndpoints()');
+    }
+    zeniAPIClientState_1.zeniAPI.apiEndPoints = restEndPoints;
+}
 /**
  * Returns current Zeni REST API client config.
  * @returns
@@ -110,7 +124,6 @@ function zeniAPIClientConfig() {
  * @param tenantId tenant id
  */
 async function initializeWithNewStore(restEndPoints, userId, sessionId, tenantId) {
-    console.info(`initializeWithNewStore with ${JSON.stringify(restEndPoints)}`);
     initialize3rdPartyExtensions();
     const zeniHttpClient = new zeniAPI_1.ZeniAPIClient(restEndPoints, userId, sessionId, tenantId);
     const store = (0, configureStore_1.default)(zeniHttpClient, false);

package/lib/reducer.d.ts

@@ -165,7 +165,6 @@ declare const reducers: import("redux").Reducer<import("redux").CombinedState<{
     taskGroupTemplateViewState: import("./view/taskManager/taskGroupTemplateView/taskGroupTemplateViewState").TaskGroupTemplateViewState;
     taskGroupViewState: import("./view/taskManager/taskGroupView/taskGroupViewState").TaskGroupViewState;
     taskListState: import("./view/taskManager/taskListView/taskList").TaskListState;
-    timeEntryViewState: import("./view/taskManager/timeEntryView/timeEntryViewState").TimeEntryViewState;
     tagViewState: import(".").TagViewState;
     tasksCardState: import("./view/tasksCard/tasksCardState").TasksCardState;
     topExState: import("./view/topEx/topExState").TopExState;
@@ -266,7 +265,6 @@ declare const reducers: import("redux").Reducer<import("redux").CombinedState<{
     subscriptionState: import("./entity/subscription/subscription/subscriptionState").SubscriptionState;
     subscriptionSummaryState: import("./entity/subscription/subscriptionSummary/subscriptionSummaryState").SubscriptionSummaryState;
     tagState: import(".").TagState;
-    timeEntryState: import("./entity/timeEntry/timeEntryState").TimeEntryState;
     taskGroupState: import(".").TaskGroupState;
     taskGroupTemplateState: import("./entity/taskGroupTemplate/taskGroupTemplateState").TaskGroupTemplateState;
     taskState: import("./entity/task/taskState").TaskState;

package/lib/reducer.js

@@ -109,7 +109,6 @@ const taskGroupReducer_1 = __importStar(require("./entity/taskGroup/taskGroupRed
 const taskGroupTemplateReducer_1 = __importStar(require("./entity/taskGroupTemplate/taskGroupTemplateReducer"));
 const taskSummaryReducer_1 = __importStar(require("./entity/taskSummary/taskSummaryReducer"));
 const tenantReducer_1 = __importStar(require("./entity/tenant/tenantReducer"));
-const timeEntryReducer_1 = __importStar(require("./entity/timeEntry/timeEntryReducer"));
 const toastNotificationReducer_1 = __importStar(require("./entity/toastNotification/toastNotificationReducer"));
 const transactionReducer_1 = __importStar(require("./entity/transaction/transactionReducer"));
 const transactionActivityLogReducer_1 = __importStar(require("./entity/transactionActivityLog/transactionActivityLogReducer"));
@@ -295,7 +294,6 @@ const taskDetailReducer_1 = __importStar(require("./view/taskManager/taskDetailV
 const taskGroupTemplateViewReducer_1 = __importStar(require("./view/taskManager/taskGroupTemplateView/taskGroupTemplateViewReducer"));
 const taskGroupViewReducer_1 = __importStar(require("./view/taskManager/taskGroupView/taskGroupViewReducer"));
 const taskListReducer_1 = __importStar(require("./view/taskManager/taskListView/taskListReducer"));
-const timeEntryViewReducer_1 = __importStar(require("./view/taskManager/timeEntryView/timeEntryViewReducer"));
 const tasksCardReducer_1 = __importStar(require("./view/tasksCard/tasksCardReducer"));
 const topExReducer_1 = __importStar(require("./view/topEx/topExReducer"));
 const transactionActivityLogViewReducer_1 = __importStar(require("./view/transactionActivityLogView/transactionActivityLogViewReducer"));
@@ -384,7 +382,6 @@ const initialEntitiesState = {
     subscriptionState: subscriptionReducer_1.initialState,
     subscriptionSummaryState: subscriptionSummaryReducer_1.initialState,
     tagState: tagReducer_1.initialState,
-    timeEntryState: timeEntryReducer_1.initialState,
     taskGroupState: taskGroupReducer_1.initialState,
     taskGroupTemplateState: taskGroupTemplateReducer_1.initialState,
     taskState: taskReducer_1.initialState,
@@ -566,7 +563,6 @@ const initialViewsState = {
     taskGroupTemplateViewState: taskGroupTemplateViewReducer_1.initialState,
     taskGroupViewState: taskGroupViewReducer_1.initialState,
     taskListState: taskListReducer_1.initialState,
-    timeEntryViewState: timeEntryViewReducer_1.initialState,
     tasksCardState: tasksCardReducer_1.initialState,
     topExState: topExReducer_1.initialState,
     transactionActivityLogViewState: transactionActivityLogViewReducer_1.initialState,
@@ -675,7 +671,6 @@ const entityReducers = {
     subscriptionState: subscriptionReducer_1.default,
     subscriptionSummaryState: subscriptionSummaryReducer_1.default,
     tagState: tagReducer_1.default,
-    timeEntryState: timeEntryReducer_1.default,
     taskGroupState: taskGroupReducer_1.default,
     taskGroupTemplateState: taskGroupTemplateReducer_1.default,
     taskState: taskReducer_1.default,
@@ -856,7 +851,6 @@ const viewReducers = {
     taskGroupTemplateViewState: taskGroupTemplateViewReducer_1.default,
     taskGroupViewState: taskGroupViewReducer_1.default,
     taskListState: taskListReducer_1.default,
-    timeEntryViewState: timeEntryViewReducer_1.default,
     tagViewState: tagViewReducer_1.default,
     tasksCardState: tasksCardReducer_1.default,
     topExState: topExReducer_1.default,

package/lib/rootStateTypes.d.ts

@@ -69,7 +69,6 @@ import { TaskGroupState } from './entity/taskGroup/taskGroupState';
 import { TaskGroupTemplateState } from './entity/taskGroupTemplate/taskGroupTemplateState';
 import { TaskSummaryState } from './entity/taskSummary/taskSummaryState';
 import { TenantState } from './entity/tenant/tenantState';
-import { TimeEntryState } from './entity/timeEntry/timeEntryState';
 import { ToastNotificationState } from './entity/toastNotification/toastNotificationState';
 import { TransactionState } from './entity/transaction/transactionState';
 import { TransactionActivityLogState } from './entity/transactionActivityLog/transactionActivityLogState';
@@ -253,7 +252,6 @@ import { TaskDetailViewState } from './view/taskManager/taskDetailView/taskDetai
 import { TaskGroupTemplateViewState } from './view/taskManager/taskGroupTemplateView/taskGroupTemplateViewState';
 import { TaskGroupViewState } from './view/taskManager/taskGroupView/taskGroupViewState';
 import { TaskListState } from './view/taskManager/taskListView/taskList';
-import { TimeEntryViewState } from './view/taskManager/timeEntryView/timeEntryViewState';
 import { TasksCardState } from './view/tasksCard/tasksCardState';
 import { TopExState } from './view/topEx/topExState';
 import { TransactionActivityLogViewState } from './view/transactionActivityLogView/transactionActivityLogViewState';
@@ -346,7 +344,6 @@ export type EntitiesState = {
     taskState: TaskState;
     taskSummaryState: TaskSummaryState;
     tenantState: TenantState;
-    timeEntryState: TimeEntryState;
     toastNotificationState: ToastNotificationState;
     transactionActivityLogState: TransactionActivityLogState;
     transactionState: TransactionState;
@@ -524,7 +521,6 @@ export type ViewsState = {
     taskGroupViewState: TaskGroupViewState;
     taskListState: TaskListState;
     tasksCardState: TasksCardState;
-    timeEntryViewState: TimeEntryViewState;
     topExState: TopExState;
     transactionActivityLogViewState: TransactionActivityLogViewState;
     transactionDetailState: TransactionDetailState;

package/lib/sessionEndpointsLoader.d.ts

@@ -0,0 +1,9 @@
+/**
+ * Called after sign-in, before the user's tenants are fetched. The app uses it
+ * to fetch the endpoints that are only served to a signed-in session and to
+ * hand them to `updateZeniAPIEndpoints`. Without a loader, tenants are fetched
+ * straight away with the endpoints given to `initialize`.
+ */
+export type SessionEndpointsLoader = (userId: string, sessionId?: string) => Promise<void>;
+export declare function setSessionEndpointsLoader(loader: SessionEndpointsLoader | undefined): void;
+export declare function getSessionEndpointsLoader(): SessionEndpointsLoader | undefined;

package/lib/sessionEndpointsLoader.js

@@ -0,0 +1,11 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.setSessionEndpointsLoader = setSessionEndpointsLoader;
+exports.getSessionEndpointsLoader = getSessionEndpointsLoader;
+let sessionEndpointsLoader;
+function setSessionEndpointsLoader(loader) {
+    sessionEndpointsLoader = loader;
+}
+function getSessionEndpointsLoader() {
+    return sessionEndpointsLoader;
+}

package/lib/view/taskManager/taskDetailView/taskDetailSelector.d.ts

@@ -25,10 +25,6 @@ export interface TaskDetailSelectorView extends SelectorView {
     isDeleted?: boolean;
     recurringSourceTaskId?: ID;
     snoozedUntil?: ZeniDate | null;
-    /** Drives the Category control in the For Internal Use block. Read from
-     *  the task entity rather than mirrored into local form data: it is not
-     *  editable inline, and a mirror would need its own discard path. */
-    taskCategoryId?: ID;
     taskGroupId?: ID;
 }
 export declare const getTaskDetail: (state: RootState, taskId?: ID) => TaskDetailSelectorView;

package/lib/view/taskManager/taskDetailView/taskDetailSelector.js

@@ -32,7 +32,6 @@ const getTaskDetail = (state, taskId) => {
     let isDeleted = undefined;
     let depth = undefined;
     let recurringSourceTaskId = undefined;
-    let taskCategoryId = undefined;
     let snoozedUntil = undefined;
     let taskGroupId = undefined;
     let subtasks = [];
@@ -46,7 +45,6 @@ const getTaskDetail = (state, taskId) => {
         if (taskEntity != null) {
             createdByUser = (0, userAndRole_1.getUserAndUserRole)(userState, userRoleState, addressState, taskEntity.createdBy);
             depth = taskEntity.depth;
-            taskCategoryId = taskEntity.taskCategoryId;
             isArchived = taskEntity.isArchived;
             isDeleted = taskEntity.isDeleted;
             recurringSourceTaskId = taskEntity.recurringSourceTaskId;
@@ -111,7 +109,6 @@ const getTaskDetail = (state, taskId) => {
         version: 0,
         createdByUser,
         depth,
-        taskCategoryId,
         isArchived,
         isDeleted,
         recurringSourceTaskId,

package/lib/view/taskManager/taskListView/taskList.d.ts

@@ -48,7 +48,7 @@ export interface TaskFilterCategory {
     values: (string | ZeniDate)[];
     valuesCombinationOperator: 'ANY';
 }
-export type TaskListFilterCategoryField = 'status' | 'dueDate' | 'assignee' | 'createdBy' | 'creationDate' | 'priority' | 'tag';
+export type TaskListFilterCategoryField = 'status' | 'dueDate' | 'assignee' | 'creationDate' | 'priority' | 'tag';
 export declare const TASK_LIST_GROUP_BY_KEYS: readonly ["groupName", "status", "dueDate", "assignee", "priority", "tag"];
 export declare const toTaskListGroupByKeyType: (v: string) => "status" | "dueDate" | "priority" | "assignee" | "tag" | "groupName";
 export declare const toTaskListGroupByKeyTypeStrict: (v: string) => "status" | "dueDate" | "priority" | "assignee" | "tag" | "groupName" | undefined;

package/lib/view/taskManager/taskListView/taskList.js

@@ -53,11 +53,6 @@ exports.TASK_LIST_FILTER_CATEGORIES = [
         type: 'user',
         multiple: true,
     },
-    {
-        value: 'createdBy',
-        type: 'user',
-        multiple: true,
-    },
     { value: 'creationDate', type: 'dateRange', multiple: true },
 ];
 exports.TASK_LIST_GROUP_BY_CATEGORIES = [

package/lib/view/taskManager/taskListView/taskListFilterHelpers.js

@@ -15,8 +15,6 @@ const getCategoryValueForTaskList = (key, task) => {
             return task.createTime;
         case 'assignee':
             return task.assignees;
-        case 'createdBy':
-            return task.createdBy;
         default:
             return undefined;
     }

package/lib/view/taskManager/taskListView/taskListViewHelpers.d.ts

@@ -3,7 +3,7 @@ import { Task } from '../../../entity/task/taskState';
 import { User } from '../../../entity/user/userState';
 import { ZeniDate } from '../../../zeniDayJS';
 import { TaskGroupKey } from './taskList';
-export type SubtaskSortKey = 'name' | 'status' | 'assignee' | 'tag' | 'priority' | 'dueDate' | 'timeSpent';
+export type SubtaskSortKey = 'name' | 'status' | 'assignee' | 'tag';
 export type SubtaskSortOrder = 'asc' | 'desc';
 export declare const getDueDateValueFromDueDateGroupId: (groupId: string) => ZeniDate;
 export declare const getTaskUpdates: (groupByKey: TaskGroupKey, groupId: string) => Partial<Task>;

package/lib/view/taskManager/taskListView/taskListViewHelpers.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.sortSubtasks = exports.getTaskUpdates = exports.getDueDateValueFromDueDateGroupId = void 0;
-const formatMinutesToFromHHMM_1 = require("../../../commonStateTypes/fiscalYearHelpers/formatMinutesToFromHHMM");
 const zeniDayJS_1 = require("../../../zeniDayJS");
 const taskDetailSelector_1 = require("../taskDetailView/taskDetailSelector");
 const taskList_1 = require("./taskList");
@@ -69,11 +68,6 @@ const sortSubtasks = (subtasks, sortKey, sortOrder, userList, tagList) => {
         return acc;
     }, {});
     const unknownStatusRank = taskDetailSelector_1.allTaskStatus.length;
-    const priorityOrder = taskDetailSelector_1.allTaskPriority.reduce((acc, priority, index) => {
-        acc[priority.code] = index;
-        return acc;
-    }, {});
-    const unknownPriorityRank = taskDetailSelector_1.allTaskPriority.length;
     const cmpStr = (a, b) => a.localeCompare(b, undefined, { sensitivity: 'base' });
     const keyFn = (task) => {
         if (sortKey === 'name') {
@@ -100,37 +94,6 @@ const sortSubtasks = (subtasks, sortKey, sortOrder, userList, tagList) => {
             const tag = tagList.find((candidate) => candidate.tagId === firstTagId);
             return tag?.name ?? '';
         }
-        if (sortKey === 'priority') {
-            // Rank by the canonical order, not the code string: alphabetical would
-            // sort "high" before "urgent" and read as wrong to anyone scanning it.
-            return priorityOrder[task.priority?.code] ?? unknownPriorityRank;
-        }
-        if (sortKey === 'dueDate') {
-            // A task with no date is not "earliest", so undated rows belong at the
-            // bottom in BOTH directions. The comparator negates for 'desc', so one
-            // fixed sentinel would flip them to the top — the sentinel has to flip
-            // with the order to survive that negation.
-            // `dueDate` is already a ZeniDate (dayjs), so valueOf() is the epoch.
-            // An unparseable date yields NaN, and a NaN comparator leaves the array
-            // untouched -- the same silent failure the timeSpent branch guards.
-            // Treat it as undated rather than as a date.
-            const due = task.dueDate;
-            const epoch = due?.valueOf();
-            if (epoch == null || Number.isNaN(epoch)) {
-                return sortOrder === 'asc'
-                    ? Number.MAX_SAFE_INTEGER
-                    : Number.MIN_SAFE_INTEGER;
-            }
-            return epoch;
-        }
-        if (sortKey === 'timeSpent') {
-            // `timeSpent` is an "HHMM" STRING (no colon). Comparing it as text puts
-            // "1000" before "0900" only by luck of padding, and any malformed value
-            // makes the converter return NaN -- which a comparator turns into "no
-            // sort at all", silently. Coerce that to 0.
-            const minutes = (0, formatMinutesToFromHHMM_1.convertHHMMStrToMinutes)(task.timeSpent ?? '');
-            return Number.isFinite(minutes) ? minutes : 0;
-        }
         return '';
     };
     const copy = [...subtasks];

package/lib/view/transactionDetail/createTransaction/createTransactionPayload.d.ts

@@ -12,8 +12,9 @@ import { CreateTransactionLocalData } from './createTransactionState';
  * 1. `line_detail_type` is snake_case. The PascalCase strings in the backend are Python
  *    constant NAMES; ZENI_LINE_DETAIL_TYPE_TO_QBO_LINE_DETAIL_TYPE_MAP converts them for the
  *    body the backend sends on to QuickBooks, never for ours.
- * 2. Only the QBO Purchases (check, credit_card_credit) send a reference number, as
- *    `document_number`. A journal entry sends none.
+ * 2. The QBO Purchases (check, credit_card_credit) send their reference number as
+ *    `document_number`; a journal entry sends its entry name as `document_id`. Both land on
+ *    QBO's DocNumber, but QBOJournalEntry reads only `document_id`.
  */
 export interface CreateTransactionAccountPayload {
     qbo_id: ID;
@@ -25,14 +26,15 @@ export interface CreateTransactionLineDetailPayload {
         qbo_id: ID;
     };
     /**
-     * journal_entry only, and it MUST sit inside line_detail. The write validator's docstring
-     * warns that a misplaced posting_type is silently dropped and then rejected by QBO.
+     * journal_entry only, inside line_detail. QBOJournalEntry reads only posting_type_name; a
+     * lowercase posting_type is dropped and QBO then rejects the line.
      */
-    posting_type?: 'credit' | 'debit';
+    posting_type_name?: 'Credit' | 'Debit';
     /** Same shape the transaction update sends on a vendor line. */
     project?: ProjectBasePayload;
+    /** journal_entry only. The backend reads the QBO vendor id from `vendor_id`, not `qbo_id`. */
     vendor?: {
-        qbo_id: ID;
+        vendor_id: ID;
     };
 }
 export interface CreateTransactionLinePayload {
@@ -51,6 +53,8 @@ export interface CreateTransactionRemitToAddressPayload {
 export interface CreateTransactionRequest extends Record<string, unknown> {
     transaction_date: string;
     transaction_type: string;
+    /** journal_entry entry name (QBO DocNumber). */
+    document_id?: string;
     /** check / credit_card_credit reference number (QBO Purchase). */
     document_number?: string;
     from_account_id?: ID;

package/lib/view/transactionDetail/createTransaction/createTransactionPayload.js

@@ -7,6 +7,10 @@ const classSelector_1 = require("../../../entity/class/classSelector");
 const projectPayload_1 = require("../../../entity/project/projectPayload");
 const projectSelector_1 = require("../../../entity/project/projectSelector");
 const vendorSelector_1 = require("../../../entity/vendor/vendorSelector");
+const POSTING_TYPE_NAME = {
+    credit: 'Credit',
+    debit: 'Debit',
+};
 const toRemitToAddressPayload = (address) => {
     if (address == null) {
         return undefined;
@@ -38,10 +42,12 @@ const toLinePayload = (state, line, transactionType) => {
         });
     }
     if (transactionType === 'journal_entry') {
-        lineDetail.posting_type = line.postingType;
+        if (line.postingType != null) {
+            lineDetail.posting_type_name = POSTING_TYPE_NAME[line.postingType];
+        }
         const vendorQboId = toQboLineVendorId(state, line.vendorQboId);
         if (vendorQboId != null) {
-            lineDetail.vendor = { qbo_id: vendorQboId };
+            lineDetail.vendor = { vendor_id: vendorQboId };
         }
     }
     return {
@@ -132,6 +138,10 @@ const prepareCreateTransactionPayload = (state, localData) => {
     if (transactionType === 'journal_entry') {
         return {
             ...base,
+            ...(localData.documentNumber != null &&
+                localData.documentNumber !== '' && {
+                document_id: localData.documentNumber,
+            }),
             lines,
         };
     }
@@ -145,6 +155,8 @@ const prepareCreateTransactionPayload = (state, localData) => {
     if (transactionType === 'check') {
         return {
             ...purchaseBase,
+            // The form has no payment method field for a check; qbo_purchase.py still requires it.
+            payment_type: 'check',
             document_number: localData.documentNumber,
             print_status: localData.printStatus,
             remit_to_address: toRemitToAddressPayload(localData.remitToAddress),

package/lib/view/transactionDetail/createTransaction/createTransactionSelector.d.ts

@@ -22,7 +22,10 @@ export interface CreateTransactionView extends SelectorView {
     lineAccountListFetchState: FetchState;
     localData: CreateTransactionLocalData;
     paymentAccountListFetchState: FetchState;
-    /** Bank, credit card and other current asset accounts for the header payment account. */
+    /**
+     * Bank, credit card and other current asset accounts for the header payment account; bank
+     * accounts only for a check.
+     */
     paymentAccountListNestedAccountHierarchy: NestedAccountHierarchyForReport[];
     transactionType: CreatableTransactionType;
     /** Populated for transfer only; the COA would offer ineligible accounts. */

package/lib/view/transactionDetail/createTransaction/createTransactionSelector.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getCreateTransactionView = exports.getCreateTransactionVendorRecommendations = exports.getCreateTransactionStatus = exports.getCreateTransactionLocalData = void 0;
 const toolkit_1 = require("@reduxjs/toolkit");
+const accountSelector_1 = require("../../../entity/account/accountSelector");
 const accountListSelector_1 = require("../../accountList/accountListSelector");
 const classListSelector_1 = require("../../classList/classListSelector");
 const createTransferEntrySelector_1 = require("../../createTransferEntry/createTransferEntrySelector");
@@ -28,6 +29,23 @@ const getCreateTransactionVendorRecommendations = (state, vendorQboId) => {
     };
 };
 exports.getCreateTransactionVendorRecommendations = getCreateTransactionVendorRecommendations;
+/**
+ * QBO sub-accounts share their parent's type, so filtering the root nodes keeps whole subtrees.
+ *
+ * Memoized on its own: createTransactionsState changes on every keystroke, which re-runs the view
+ * projection, and the filter would otherwise hand the dropdown a new array each time.
+ */
+const getPaymentAccountListNestedAccountHierarchy = (0, toolkit_1.createSelector)([
+    (state) => (0, accountListSelector_1.getNestedAccountListHierarchy)(state.accountListState, 'createTransactionPaymentAccountList'),
+    (state) => state.accountState,
+    (_state, transactionType) => transactionType,
+], (hierarchy, accountState, transactionType) => {
+    if (transactionType !== 'check') {
+        return hierarchy;
+    }
+    return hierarchy.filter((node) => (0, accountSelector_1.getAccountBase)(accountState, node.accountId[0], 'account_list')
+        ?.accountType === 'bank');
+});
 /**
  * Memoized: app-ui reads this straight from useSelector, and the vendor list and hierarchies are
  * rebuilt on every call, so an unmemoized version re-renders the page on every store action.
@@ -36,10 +54,11 @@ exports.getCreateTransactionView = (0, toolkit_1.createSelector)([
     (state) => state.transactionDetailState.createTransactionsState,
     (state) => state.vendorState.vendorByID,
     (state) => state.accountListState,
+    getPaymentAccountListNestedAccountHierarchy,
     (state) => state.classListState,
     (state) => (0, createTransferEntrySelector_1.getResolvedTransferAccounts)(state).transferAccountsList,
     (_state, transactionType) => transactionType,
-], (createTransactionsState, vendorByID, accountListState, classListState, transferAccountsList, transactionType) => ({
+], (createTransactionsState, vendorByID, accountListState, paymentAccountListNestedAccountHierarchy, classListState, transferAccountsList, transactionType) => ({
     transactionType,
     localData: createTransactionsState.newTransactionState[transactionType],
     createStatus: createTransactionsState.createStatus,
@@ -49,7 +68,7 @@ exports.getCreateTransactionView = (0, toolkit_1.createSelector)([
     lineAccountListFetchState: (0, accountListSelector_1.getAccountListFetchState)(accountListState, 'createTransactionLineAccountList'),
     paymentAccountListFetchState: (0, accountListSelector_1.getAccountListFetchState)(accountListState, 'createTransactionPaymentAccountList'),
     accountListNestedAccountHierarchy: (0, accountListSelector_1.getNestedAccountListHierarchy)(accountListState, 'createTransactionLineAccountList'),
-    paymentAccountListNestedAccountHierarchy: (0, accountListSelector_1.getNestedAccountListHierarchy)(accountListState, 'createTransactionPaymentAccountList'),
+    paymentAccountListNestedAccountHierarchy,
     classListNestedAccountHierarchy: (0, classListSelector_1.getNestedClassListHierarchy)(classListState),
     transferAccountsList: transactionType === 'transfer' ? transferAccountsList : [],
     fetchState: createTransactionsState.createStatus.fetchState,

package/lib/view/transactionDetail/createTransaction/createTransactionState.d.ts

@@ -54,7 +54,7 @@ export interface CreateTransactionLocalData {
     /** Empty for transfer, which has no lines at all. */
     lines: CreateTransactionLineLocalData[];
     transactionType: CreatableTransactionType;
-    /** check / credit_card_credit reference number. */
+    /** check / credit_card_credit reference number, or the journal_entry entry name. */
     documentNumber?: string;
     /** transfer */
     fromAccountId?: ID;

package/lib/zeniAPI.d.ts

@@ -25,7 +25,7 @@ export interface RESTAPIEndpoints {
     readonly tenantMicroServiceBaseUrl: string;
 }
 export interface ZeniAPI<ResponseType = FetchResponse> {
-    readonly apiEndPoints: RESTAPIEndpoints;
+    apiEndPoints: RESTAPIEndpoints;
     readonly sessionId?: string;
     readonly tenantId?: string;
     readonly userId?: string;
@@ -40,7 +40,7 @@ export interface ZeniAPI<ResponseType = FetchResponse> {
     update(userId?: string, sessionId?: string, tenantId?: string): void;
 }
 export declare class ZeniAPIClient implements ZeniAPI {
-    readonly apiEndPoints: RESTAPIEndpoints;
+    apiEndPoints: RESTAPIEndpoints;
     userId?: string | undefined;
     sessionId?: string | undefined;
     tenantId?: string | undefined;

package/package.json

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

package/lib/entity/timeEntry/timeEntryPayload.d.ts

@@ -1,26 +0,0 @@
-import { ID } from '../../commonStateTypes/common';
-/** Exactly what `task-manager` returns. snake_case, API-nullable. */
-export interface TimeEntryPayload {
-    create_time: string;
-    entry_date: string;
-    minutes: number;
-    source: string;
-    task_id: ID;
-    time_entry_id: ID;
-    update_time: string;
-    user_id: ID;
-    /** Derived from the category server-side (D15/D22) and READ-ONLY. Null when
-     *  the task has no category, or the category is not yet classified. */
-    billing_type?: string | null;
-    created_by?: ID | null;
-    department?: string | null;
-    task_category_id?: ID | null;
-}
-export interface TaskCategoryPayload {
-    is_active: boolean;
-    name: string;
-    task_category_id: ID;
-    billing_type?: string | null;
-    service?: string | null;
-    vertical?: string | null;
-}

package/lib/entity/timeEntry/timeEntryPayload.js

@@ -1,2 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });

package/lib/entity/timeEntry/timeEntryReducer.d.ts

@@ -1,8 +0,0 @@
-import { TaskCategoryPayload, TimeEntryPayload } from './timeEntryPayload';
-import { TaskCategory, TimeEntry, TimeEntryState } from './timeEntryState';
-export declare const initialState: TimeEntryState;
-export declare const updateTimeEntries: import("@reduxjs/toolkit").ActionCreatorWithPayload<TimeEntryPayload[], "timeEntry/updateTimeEntries">, removeTimeEntry: import("@reduxjs/toolkit").ActionCreatorWithPayload<string, "timeEntry/removeTimeEntry">, updateTaskCategories: import("@reduxjs/toolkit").ActionCreatorWithPayload<TaskCategoryPayload[], "timeEntry/updateTaskCategories">;
-declare const _default: import("redux").Reducer<TimeEntryState>;
-export default _default;
-export declare function mapTimeEntryPayloadToTimeEntry(payload: TimeEntryPayload): TimeEntry;
-export declare function mapTaskCategoryPayloadToTaskCategory(payload: TaskCategoryPayload): TaskCategory;