Simplify workflow timing fields

Signed-off-by: Taylor Ho <taylorkmho@gmail.com>
This commit is contained in:
Taylor Ho
2026-08-14 10:43:59 -07:00
parent 37ba923007
commit 3fd80977f6
@@ -254,7 +254,7 @@ function StepConfigFields({
disabled={disabled}
fallbackSeconds={DEFAULT_APPROVAL_TIMEOUT_SECONDS}
id={`${prefix}-timeout`}
label="Timeout (optional)"
label="Timeout"
onChange={(timeout) => onUpdate({ ...step, timeout })}
placeholder="24h"
value={step.timeout ?? ""}
@@ -299,10 +299,6 @@ function StepConfigFields({
}
}
function SectionHeading({ title }: { title: string }) {
return <h4 className="text-sm font-semibold text-foreground">{title}</h4>;
}
export function WorkflowStepCard({
bare = false,
channels = [],
@@ -356,7 +352,7 @@ export function WorkflowStepCard({
</div>
) : null}
<section className="space-y-4 pb-5">
<section className="space-y-5 pb-5">
<StepConfigFields
channels={channels}
disabled={disabled}
@@ -367,28 +363,18 @@ export function WorkflowStepCard({
triggerType={triggerType}
workflowChannelId={workflowChannelId}
/>
</section>
<section className="space-y-4 border-t border-border/50 py-5">
<SectionHeading title="Step timeout" />
<div className="space-y-2">
<WorkflowDurationField
disabled={disabled}
fallbackSeconds={DEFAULT_STEP_TIMEOUT_SECONDS}
id={`${prefix}-timeout-secs`}
label="Timeout (optional)"
onChange={(timeoutSecs) => onUpdate({ ...step, timeoutSecs })}
placeholder="5m"
value={step.timeoutSecs ?? ""}
/>
<p className="text-xs text-muted-foreground">
Defaults to the workflow timeout.
</p>
</div>
<WorkflowDurationField
disabled={disabled}
fallbackSeconds={DEFAULT_STEP_TIMEOUT_SECONDS}
id={`${prefix}-timeout-secs`}
label="Timeout"
onChange={(timeoutSecs) => onUpdate({ ...step, timeoutSecs })}
placeholder="5m"
value={step.timeoutSecs ?? ""}
/>
</section>
<section className="space-y-4 border-t border-border/50 pt-5">
<SectionHeading title="Step details" />
<div className="space-y-1.5">
<FieldLabel htmlFor={`${prefix}-name`}>Name (optional)</FieldLabel>
<Input