Skip to content

feat: exclude TTL deployments from route create deployment dropdown#81

Merged
acoshift merged 1 commit into
mainfrom
claude/competent-raman-eb9bd4
May 12, 2026
Merged

feat: exclude TTL deployments from route create deployment dropdown#81
acoshift merged 1 commit into
mainfrom
claude/competent-raman-eb9bd4

Conversation

@acoshift
Copy link
Copy Markdown
Member

Summary

  • Filters out deployments with a TTL set (ttl !== 0) from the deployment dropdown on the route create page
  • TTL deployments are temporary (auto-deleted after a period) or expired (ttl === -1, pending deletion), so they should not be valid route targets

Reviewer notes

The filter x.ttl === 0 aligns with the existing convention used throughout the codebase: ttl === 0 = permanent, ttl > 0 = has TTL, ttl === -1 = expired/pending deletion.

Deployments with a TTL set are temporary and should not be used as
route targets. Filter them out (ttl !== 0) from the dropdown on the
route create page.

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: 8f399ac
Status: ✅  Deploy successful!
Preview URL: https://680ffb3a.deploys-app--console.pages.dev
Branch Preview URL: https://claude-competent-raman-eb9bd.deploys-app--console.pages.dev

View logs

@acoshift acoshift merged commit 03bb2e2 into main May 12, 2026
3 checks passed
@acoshift acoshift deleted the claude/competent-raman-eb9bd4 branch May 12, 2026 07:50
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