diff --git a/client/src/components/jobs-detail-header/jobs-detail-header.component.jsx b/client/src/components/jobs-detail-header/jobs-detail-header.component.jsx index a3a0bd89f..4bd4774f9 100644 --- a/client/src/components/jobs-detail-header/jobs-detail-header.component.jsx +++ b/client/src/components/jobs-detail-header/jobs-detail-header.component.jsx @@ -81,16 +81,17 @@ export function JobsDetailHeader({ job, bodyshop, disabled, insertAuditTrail, is const employeeData = bodyshop.associations.find((a) => a.useremail === job.admin_clerk)?.user?.employee ?? null; // Handle checkbox changes - const handleCheckboxChange = async (field, checked) => { + const handleCheckboxChange = async (field, e) => { + e.preventDefault(); + e.stopPropagation(); + const checked = e.target.checked; const value = checked ? dayjs().toISOString() : null; try { const ret = await updateJob({ variables: { jobId: job.id, job: { [field]: value } - }, - refetchQueries: ["GET_JOB_BY_PK"], - awaitRefetchQueries: true + } }); insertAuditTrail({ jobid: job.id, @@ -182,7 +183,7 @@ export function JobsDetailHeader({ job, bodyshop, disabled, insertAuditTrail, is handleCheckboxChange("estimate_sent_approval", e.target.checked)} + onChange={(e) => handleCheckboxChange("estimate_sent_approval", e)} disabled={disabled || isPartsEntry} > {job.estimate_sent_approval && ( @@ -197,7 +198,7 @@ export function JobsDetailHeader({ job, bodyshop, disabled, insertAuditTrail, is handleCheckboxChange("estimate_approved", e.target.checked)} + onChange={(e) => handleCheckboxChange("estimate_approved", e)} disabled={disabled || isPartsEntry} > {job.estimate_approved && ( diff --git a/client/src/components/task-list/task-list.container.jsx b/client/src/components/task-list/task-list.container.jsx index a0d0a79c7..238c113d3 100644 --- a/client/src/components/task-list/task-list.container.jsx +++ b/client/src/components/task-list/task-list.container.jsx @@ -33,8 +33,7 @@ export function TaskListContainer({ currentUser, onlyMine, parentJobId, - showRo = true, - disableJobRefetch = false + showRo = true }) { const { t } = useTranslation(); const notification = useNotification(); @@ -91,10 +90,6 @@ export function TaskListContainer({ refetchQueries: [Object.keys(query)[0]] }; - if (!disableJobRefetch) { - toggledTaskObject.refetchQueries.push("GET_JOB_BY_PK"); - } - const toggledTask = await toggleTaskCompleted(toggledTaskObject); if (!toggledTask.errors) { @@ -144,10 +139,6 @@ export function TaskListContainer({ refetchQueries: [Object.keys(query)[0]] }; - if (!disableJobRefetch) { - toggledTaskObject.refetchQueries.push("GET_JOB_BY_PK"); - } - const toggledTask = await toggleTaskDeleted(toggledTaskObject); if (!toggledTask.errors) { diff --git a/client/src/pages/tasks/tasks.page.component.jsx b/client/src/pages/tasks/tasks.page.component.jsx index 80d02aba8..6c5edc1c9 100644 --- a/client/src/pages/tasks/tasks.page.component.jsx +++ b/client/src/pages/tasks/tasks.page.component.jsx @@ -23,17 +23,10 @@ export function TasksPageComponent({ bodyshop, currentUser, type }) { relationshipType={"assigned_to"} query={{ QUERY_MY_TASKS_PAGINATED }} titleTranslation={"tasks.titles.my_tasks"} - disableJobRefetch={true} /> ); case taskPageTypes.ALL_TASKS: - return ( - - ); + return ; default: return <>; }