From 96dc2726232fc02c836b29968550d7dc5af03e36 Mon Sep 17 00:00:00 2001 From: kshitijk4poor <82637225+kshitijk4poor@users.noreply.github.com> Date: Sat, 9 May 2026 13:40:57 +0530 Subject: [PATCH] fix(cron): use getJobState helper in handlePauseResume MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Self-review follow-up: handlePauseResume read job.state directly while the rest of the page goes through getJobState(), which falls back to the enabled flag when state is null/undefined. With the backend normalizer in this PR, state is always populated on the wire, so this has no observable effect today — but using the helper keeps the page consistent and resilient against older Hermes backends that don't run the normalizer. --- web/src/pages/CronPage.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/src/pages/CronPage.tsx b/web/src/pages/CronPage.tsx index 7234007b10a..e994c96f270 100644 --- a/web/src/pages/CronPage.tsx +++ b/web/src/pages/CronPage.tsx @@ -128,7 +128,7 @@ export default function CronPage() { const handlePauseResume = async (job: CronJob) => { try { - const isPaused = job.state === "paused"; + const isPaused = getJobState(job) === "paused"; if (isPaused) { await api.resumeCronJob(job.id); showToast(