Show deployments in the pull request pane with a one-click preview link #8258
Replies: 1 comment
|
Update: the prototype now supports multiple preview environments.
(Real data: vercel/commerce#1535, which deploys two Vercel environments per commit. Verified end-to-end — clicking a menu entry opens that environment's URL, and the single-environment case still opens its live preview directly.) Also hardened after a review pass: the deployments read uses the API's full page window so a heavily-redeployed environment can't crowd others out, a live deployment with no address no longer falls back to opening its build log from a button that says "Open preview" (the log fallback now applies only to non-successful deployments, where logs are what you want), and the whole read remains one budgeted GraphQL call that degrades silently rather than ever failing the PR detail view. |

Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
When a provider like Vercel deploys a preview for a pull request, the link ends up buried in the PR pane: it flows through the commit statuses into the Checks section as a generic row, and its "Details" button looks exactly like every lint and test check. Opening the deployed preview — usually the thing you actually want mid-review — takes hunting through check rows.
Proposal
environmentUrl), keeping the latest deployment per environment.deploymentsfield to the pull request detail contract, following the existing pattern for host-conditional fields (absent means "not supported / could not ask", not "deploys nowhere").Smallest useful scope
Before / after
Before — the Vercel preview is two generic "Passed" rows inside Checks:
After — a Deployments section names the environment and its state:
And the header gains a one-click "Open preview" globe button:
(Screenshots are from a working prototype running against a real PR — t3-oss/create-t3-app#2243 with a live Vercel preview. Implementation is contracts + server + web, ~600 lines including tests; happy to open a PR if this is welcome.)
Related
statusCheckRollup.targetUrlas a generic check "Details" link; this proposal only promotes deployments to a first-class, clearly-labeled affordance.All reactions