Skip to content

Commit 15ef547

Browse files
kartik-212004anikdhabalPallava-Joshi
authored
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 <[email protected]> Co-authored-by: Pallav <[email protected]>
1 parent 251d29f commit 15ef547

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

packages/features/eventtypes/components/tabs/advanced/RequiresConfirmationController.tsx

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ export default function RequiresConfirmationController({
6060
if (!requiresConfirmation) {
6161
formMethods.setValue("metadata.requiresConfirmationThreshold", undefined, { shouldDirty: true });
6262
}
63-
// eslint-disable-next-line react-hooks/exhaustive-deps
6463
}, [requiresConfirmation]);
6564

6665
const { shouldLockDisableProps } = useLockedFieldsManager({ eventType, translate: t, formMethods });
@@ -111,10 +110,11 @@ export default function RequiresConfirmationController({
111110
LockedIcon={requiresConfirmationLockedProps.LockedIcon}
112111
onCheckedChange={(val) => {
113112
formMethods.setValue("requiresConfirmation", val, { shouldDirty: true });
114-
// If we uncheck requires confirmation, we also uncheck these checkboxes
115113
if (!val) {
116114
formMethods.setValue("requiresConfirmationWillBlockSlot", false, { shouldDirty: true });
117115
formMethods.setValue("requiresConfirmationForFreeEmail", false, { shouldDirty: true });
116+
} else {
117+
formMethods.setValue("requiresConfirmationWillBlockSlot", true, { shouldDirty: true });
118118
}
119119
onRequiresConfirmation(val);
120120
}}>
@@ -135,6 +135,9 @@ export default function RequiresConfirmationController({
135135
shouldDirty: true,
136136
});
137137
setRequiresConfirmationSetup(undefined);
138+
formMethods.setValue("requiresConfirmationWillBlockSlot", true, {
139+
shouldDirty: true,
140+
});
138141
} else if (val === "notice") {
139142
formMethods.setValue("requiresConfirmation", true, { shouldDirty: true });
140143
onRequiresConfirmation(true);
@@ -143,6 +146,9 @@ export default function RequiresConfirmationController({
143146
requiresConfirmationSetup || defaultRequiresConfirmationSetup,
144147
{ shouldDirty: true }
145148
);
149+
formMethods.setValue("requiresConfirmationWillBlockSlot", true, {
150+
shouldDirty: true,
151+
});
146152
}
147153
}}>
148154
<div

0 commit comments

Comments
 (0)