Skip to content

Commit c19ec14

Browse files
fix(table): treat manual-bypass eligibility as runnable
Refactoring eligibility into classifyEligibility split the runnable answer into two reasons: 'eligible' (deps satisfied) and 'manual-bypass' (autoRun= false group on a manual run, deps don't apply). isGroupEligible only treated 'eligible' as runnable, so a manual "Run all rows" on a single autoRun=false group filtered every row out and returned triggered: 0. Cells flashed queued from the optimistic patch then went empty when the refetch landed.
1 parent 7fcc8b6 commit c19ec14

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

apps/sim/lib/table/workflow-columns.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,8 @@ export function isGroupEligible(
9292
row: TableRow,
9393
opts?: { isManualRun?: boolean; mode?: 'all' | 'incomplete' }
9494
): boolean {
95-
return classifyEligibility(group, row, opts) === 'eligible'
95+
const reason = classifyEligibility(group, row, opts)
96+
return reason === 'eligible' || reason === 'manual-bypass'
9697
}
9798

9899
/**
@@ -142,7 +143,7 @@ export async function scheduleRunsForRows(
142143
mode: opts?.mode,
143144
})
144145
reasonCounts[reason] = (reasonCounts[reason] ?? 0) + 1
145-
if (reason !== 'eligible') continue
146+
if (reason !== 'eligible' && reason !== 'manual-bypass') continue
146147
pendingRuns.push({
147148
tableId: table.id,
148149
tableName: table.name,

0 commit comments

Comments
 (0)