Improve the Vercel & Slack integration settings pages - #15
Conversation
Source PR: triggerdotdev#4848 Source head: 97cebf5
⛔ Shipwright · BlockedRecommendation: do not merge PR #15 · Tier
Findings (2)
Fireworks usage: 19,510 input · 233 output · 19,743 total tokens · $0.0044 · 7s · 0 fix iteration(s) Open the Shipwright check for full evidence and the audit bundle. Use |
| const isUninstalling = | ||
| navigation.state === "submitting" && navigation.formData?.get("intent") === "uninstall"; | ||
|
|
||
| const { projects } = useOrganization(); |
There was a problem hiding this comment.
Shipwright · HIGH
In the Vercel no-integration branch, 'organization' is referenced in 'configurePathFor' but only 'projects' is destructured from 'useOrganization()'.
Impact: In the Vercel no-integration branch, 'organization' is referenced in 'configurePathFor' but only 'projects' is destructured from 'useOrganization()'. This causes a ReferenceError when a Vercel integration is not configured and the organization has projects.
Suggested fix: Fix the review finding before release.
What
Updated layout for the Vercel and Slack organization integration settings pages to match the rest of the settings UI. Layout & UI only — no loader/action/logic changes.
Blank states (no integration)
?alerts=true) — styled like the side-menu project switcherConnected states
SettingsContainer/SettingsSection/SettingsHeader/SettingsRowCopyableText), a sticky Configure cog action per row🤖 Generated with Claude Code
Open workspace in Conductor
Source merge-base:
97e3f0aa26e080be171fd85ed5eb3cf525e53891Source head:
97cebf51132c1cdf17e1041367759a951a9e632f