From 15ef547e41efdd694b46ba1af64d0531647ee910 Mon Sep 17 00:00:00 2001 From: Kartik <103111467+kartik-212004@users.noreply.github.com> Date: Tue, 25 Nov 2025 00:23:40 +0530 Subject: [PATCH] fix: enable block calendar slots by default for required confirmations (#25239) * fix: enable block calendar slots by default for required confirmations * remove unnecessary wasenabled --------- Co-authored-by: Anik Dhabal Babu <81948346+anikdhabal@users.noreply.github.com> Co-authored-by: Pallav <90088723+Pallava-Joshi@users.noreply.github.com> --- .../tabs/advanced/RequiresConfirmationController.tsx | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/packages/features/eventtypes/components/tabs/advanced/RequiresConfirmationController.tsx b/packages/features/eventtypes/components/tabs/advanced/RequiresConfirmationController.tsx index ac0bf0b205..c72c691528 100644 --- a/packages/features/eventtypes/components/tabs/advanced/RequiresConfirmationController.tsx +++ b/packages/features/eventtypes/components/tabs/advanced/RequiresConfirmationController.tsx @@ -60,7 +60,6 @@ export default function RequiresConfirmationController({ if (!requiresConfirmation) { formMethods.setValue("metadata.requiresConfirmationThreshold", undefined, { shouldDirty: true }); } - // eslint-disable-next-line react-hooks/exhaustive-deps }, [requiresConfirmation]); const { shouldLockDisableProps } = useLockedFieldsManager({ eventType, translate: t, formMethods }); @@ -111,10 +110,11 @@ export default function RequiresConfirmationController({ LockedIcon={requiresConfirmationLockedProps.LockedIcon} onCheckedChange={(val) => { formMethods.setValue("requiresConfirmation", val, { shouldDirty: true }); - // If we uncheck requires confirmation, we also uncheck these checkboxes if (!val) { formMethods.setValue("requiresConfirmationWillBlockSlot", false, { shouldDirty: true }); formMethods.setValue("requiresConfirmationForFreeEmail", false, { shouldDirty: true }); + } else { + formMethods.setValue("requiresConfirmationWillBlockSlot", true, { shouldDirty: true }); } onRequiresConfirmation(val); }}> @@ -135,6 +135,9 @@ export default function RequiresConfirmationController({ shouldDirty: true, }); setRequiresConfirmationSetup(undefined); + formMethods.setValue("requiresConfirmationWillBlockSlot", true, { + shouldDirty: true, + }); } else if (val === "notice") { formMethods.setValue("requiresConfirmation", true, { shouldDirty: true }); onRequiresConfirmation(true); @@ -143,6 +146,9 @@ export default function RequiresConfirmationController({ requiresConfirmationSetup || defaultRequiresConfirmationSetup, { shouldDirty: true } ); + formMethods.setValue("requiresConfirmationWillBlockSlot", true, { + shouldDirty: true, + }); } }}>