Skip to content

feat: show expired-pending-deletion state for TTL=-1 deployments#80

Merged
acoshift merged 1 commit into
mainfrom
claude/gracious-hugle-c6f530
May 12, 2026
Merged

feat: show expired-pending-deletion state for TTL=-1 deployments#80
acoshift merged 1 commit into
mainfrom
claude/gracious-hugle-c6f530

Conversation

@acoshift
Copy link
Copy Markdown
Member

Summary

  • The deployment list and get APIs now return ttl=-1 when a deployment has expired but GC hasn't cleaned it up yet.
  • Previously the UI only handled ttl > 0 (active TTL) and ttl <= 0 (no TTL), so -1 was silently treated as "no TTL" and nothing was shown.
  • This PR adds an explicit branch for ttl === -1 in both the deployment list view and the detail view, showing a danger-colored clock icon with an "Expired — pending deletion" label.

Changes

Reviewer notes

No logic change to the deploy form — it already guards TTL pre-fill with ttl > 0, so -1 is correctly ignored there.

The API now returns ttl=-1 for deployments that have expired but are
waiting for GC cleanup. Surface this in the list and detail views with
a danger-colored clock icon and "Expired — pending deletion" label.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying deploys-app--console with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7112e01
Status: ✅  Deploy successful!
Preview URL: https://b87e407b.deploys-app--console.pages.dev
Branch Preview URL: https://claude-gracious-hugle-c6f530.deploys-app--console.pages.dev

View logs

@acoshift acoshift merged commit 7c1200b into main May 12, 2026
3 checks passed
@acoshift acoshift deleted the claude/gracious-hugle-c6f530 branch May 12, 2026 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant