feat(ST-cmovhveih): remove PLANNED-blokkering in idea-row-actions, add inline Bekijk-PBI button
- Removed grillBlockedReason guard for status==='planned', enabling re-grill from PLANNED - Removed the early return for PLANNED that hid all standard buttons - Added conditional 'Bekijk <code>' button at the start of the standard button set, visible only when status==='planned' and PBI + product_id are present
This commit is contained in:
parent
cd26caa22e
commit
8211b4109d
1 changed files with 7 additions and 23 deletions
|
|
@ -61,7 +61,6 @@ export function IdeaRowActions({ idea, isDemo, onArchive }: IdeaRowActionsProps)
|
|||
// ---- Grill Me ----
|
||||
const grillBlockedReason = (() => {
|
||||
if (status === 'grilling' || status === 'planning') return 'Job loopt al'
|
||||
if (status === 'planned') return 'Idee is gepland — open de PBI'
|
||||
if (!hasProductWithRepo) return 'Idee heeft een product met repo nodig'
|
||||
if (!workerOk) return 'Geen Claude-worker actief'
|
||||
return null
|
||||
|
|
@ -125,35 +124,20 @@ export function IdeaRowActions({ idea, isDemo, onArchive }: IdeaRowActionsProps)
|
|||
})
|
||||
}
|
||||
|
||||
// PLANNED-state: kortere variant met "Bekijk PBI"-link
|
||||
if (status === 'planned' && idea.pbi && idea.product_id) {
|
||||
return (
|
||||
<div className="flex items-center gap-1.5">
|
||||
return (
|
||||
<div className="flex items-center gap-1">
|
||||
{/* Bekijk PBI — alleen zichtbaar in PLANNED */}
|
||||
{status === 'planned' && idea.pbi && idea.product_id && (
|
||||
<Button
|
||||
size="sm"
|
||||
variant="outline"
|
||||
onClick={() =>
|
||||
router.push(`/products/${idea.product_id}`)
|
||||
}
|
||||
size="sm"
|
||||
onClick={() => router.push(`/products/${idea.product_id!}`)}
|
||||
>
|
||||
Bekijk {idea.pbi.code}
|
||||
<ExternalLink className="ml-1 size-3.5" />
|
||||
</Button>
|
||||
<Button
|
||||
size="sm"
|
||||
variant="ghost"
|
||||
onClick={() => router.push(`/ideas/${idea.id}`)}
|
||||
aria-label="Open idee"
|
||||
title="Open idee"
|
||||
>
|
||||
<ArrowRight className="size-4" />
|
||||
</Button>
|
||||
</div>
|
||||
)
|
||||
}
|
||||
)}
|
||||
|
||||
return (
|
||||
<div className="flex items-center gap-1">
|
||||
{/* Grill Me */}
|
||||
<ActionButton
|
||||
label="Grill"
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue