adam/shoc-frontend-new
1
0
Fork 0
mirror of https://github.com/Sea-Haven-Industries/shoc-frontend-new.git synced 2026-10-04 18:22:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

refactor(workorders): extract slide-over open-tab effect to meet function size cap

Browse source
This commit is contained in:
Alexandre Brandizzi 2026-10-01 23:17:33 -03:00
parent f9b5d3e0ea
commit e6548ffa17
1 changed files with 24 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
src/app/(protected)/workorders/_components/detail/use-slide-over-edit-state.ts
View file

@ -77,6 +77,29 @@ function useSlideOverInitialization({
}, [row?.id, editMode]);
}
// Re-apply the requested tab on every open, including a second notification for the work order
// already on screen (same row id) where the identity effect does not re-run. Tab-only so an
// in-flight edit draft is never discarded.
function useSlideOverOpenRequestTab({
row,
editMode,
initialTab,
setTab,
openRequestKey,
}: {
row: WorkOrderTableRow | null;
editMode: boolean;
initialTab: SlideOverTab;
setTab: (tab: SlideOverTab) => void;
openRequestKey: number;
}) {
useEffect(() => {
if (!row) return;
setTab(editMode ? "info" : initialTab);
// eslint-disable-next-line react-hooks/exhaustive-deps -- openRequestKey gates each open
}, [openRequestKey]);
}
export function useSlideOverEditState({
row,
editMode,
@ -109,14 +132,7 @@ export function useSlideOverEditState({
setUnsavedIntent,
});
// Re-apply the requested tab on every open, including a second notification for the work order
// already on screen (same row id) where the identity effect above does not re-run. Tab-only so
// an in-flight edit draft is never discarded.
useEffect(() => {
if (!row) return;
setTab(editMode ? "info" : initialTab);
// eslint-disable-next-line react-hooks/exhaustive-deps -- openRequestKey gates each open
}, [openRequestKey]);
useSlideOverOpenRequestTab({ row, editMode, initialTab, setTab, openRequestKey });
const lockStatus = infoSource?.status ?? row?.status;
const isInfoLocked = isSlideOverInfoLocked(lockStatus);