Skip to content

Commit 5398cb4

Browse files
committed
Make reason form field optional in delete engine form (#5297)
Fixes: DASH-381
1 parent 2eaf773 commit 5398cb4

File tree

1 file changed

+5
-17
lines changed

1 file changed

+5
-17
lines changed

apps/dashboard/src/components/engine/engine-instances-table.tsx

Lines changed: 5 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -430,34 +430,22 @@ function DeleteSubscriptionModalContent(props: {
430430

431431
<form onSubmit={form.handleSubmit(onSubmit)}>
432432
{/* Reason */}
433-
<FormControl isRequired>
433+
<FormControl>
434434
<FormLabel className="!text-base">
435435
Please share your feedback to help us improve Engine.
436436
</FormLabel>
437437
<RadioGroup>
438438
<div className="flex flex-col gap-2">
439-
<Radio
440-
value="USING_SELF_HOSTED"
441-
{...form.register("reason", { required: true })}
442-
>
439+
<Radio value="USING_SELF_HOSTED" {...form.register("reason")}>
443440
<span className="text-sm"> Migrating to self-hosted </span>
444441
</Radio>
445-
<Radio
446-
value="TOO_EXPENSIVE"
447-
{...form.register("reason", { required: true })}
448-
>
442+
<Radio value="TOO_EXPENSIVE" {...form.register("reason")}>
449443
<span className="text-sm"> Too expensive </span>
450444
</Radio>
451-
<Radio
452-
value="MISSING_FEATURES"
453-
{...form.register("reason", { required: true })}
454-
>
445+
<Radio value="MISSING_FEATURES" {...form.register("reason")}>
455446
<span className="text-sm"> Missing features </span>
456447
</Radio>
457-
<Radio
458-
value="OTHER"
459-
{...form.register("reason", { required: true })}
460-
>
448+
<Radio value="OTHER" {...form.register("reason")}>
461449
<span className="text-sm"> Other </span>
462450
</Radio>
463451
</div>

0 commit comments

Comments
 (0)