fix(workflow-block): webhook URL never hydrates on canvas#4150
fix(workflow-block): webhook URL never hydrates on canvas#4150waleedlatif1 merged 1 commit intostagingfrom
Conversation
…k ID getTrigger() namespaces condition-gated subBlock IDs (e.g. webhookUrlDisplay → webhookUrlDisplay_github_release_published). The block card's useMemo was checking for an exact match on 'webhookUrlDisplay', which never matched. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview The Reviewed by Cursor Bugbot for commit 580b612. Configure here. |
Greptile SummaryThis PR fixes webhook URL display on canvas block cards for trigger blocks that use Confidence Score: 5/5
Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[SubBlockRow renders] --> B{subBlock.id<br/>startsWith 'webhookUrlDisplay'?}
B -- "No (e.g. 'tableId')" --> C[webhookUrlDisplayValue = null]
B -- "Yes (e.g. 'webhookUrlDisplay'<br/>or 'webhookUrlDisplay_github_release_published')" --> D{allSubBlockValues<br/>triggerPath set?}
D -- Yes --> E["URL = baseUrl + /api/webhooks/trigger/ + triggerPath"]
D -- No --> F["URL = baseUrl + /api/webhooks/trigger/ + blockId"]
E --> G[hydratedName = URL]
F --> G
G --> H[displayValue shows webhook URL on canvas]
C --> I[hydratedName skips webhookUrlDisplayValue]
I --> J[displayValue falls back to raw value or '-']
subgraph getTrigger namespacing
K["webhookUrlDisplay (bare — no selectedTriggerId condition)"] --> L["ID unchanged"]
M["webhookUrlDisplay (condition: selectedTriggerId)"] --> N["ID → webhookUrlDisplay_triggerId"]
end
Reviews (1): Last reviewed commit: "fix(block-card): webhook URL never hydra..." | Re-trigger Greptile |
Summary
-instead of the actual URLgetTrigger()namespaces condition-gated subBlock IDs (e.g.webhookUrlDisplay→webhookUrlDisplay_github_release_published), but the block card's useMemo checked for an exact match on'webhookUrlDisplay'which never matchedstartsWith('webhookUrlDisplay')instead of exact equalityTest plan
-