Skip to content

feat(status_page_custom_domain): writable primary attribute + /primary path#10

Merged
caballeto merged 2 commits into
mainfrom
feat/status-page-domain-primary
Apr 24, 2026
Merged

feat(status_page_custom_domain): writable primary attribute + /primary path#10
caballeto merged 2 commits into
mainfrom
feat/status-page-domain-primary

Conversation

@caballeto
Copy link
Copy Markdown
Member

Summary

Pairs with mono PR #267 (design 042 — status page domains).

Lets Terraform users mark a custom domain as the primary host for a status page, which the public page uses as the canonical link target.

  • `primary` attribute on `devhelm_status_page_custom_domain` becomes configurable (was previously read-only).
  • New `StatusPageDomainPrimaryPath` helper in `internal/api/paths.go`.
  • Resource calls `POST /api/v1/status-pages/{page}/domains/{id}/primary` on create/update when `primary = true`.

Test plan

  • `go build ./...`
  • `go vet ./...`
  • `go test ./internal/api/... ./internal/provider/resources/...`
  • After mono #267 merges, run the new `scripts/verify-domains/phase-5-track-b` smoke test (uses this provider) end-to-end.

Made with Cursor

…y path

Pairs with mono PR #267 (design 042 — status page domains).

Lets Terraform users mark a custom domain as the primary host for a
status page, which the public page uses as the canonical link target.

- `primary` attribute on `devhelm_status_page_custom_domain` becomes
  configurable (was previously read-only).
- New `StatusPageDomainPrimaryPath` helper.
- Resource calls `POST /api/v1/status-pages/{page}/domains/{id}/primary`
  on create/update when `primary = true`.

Made-with: Cursor
Picks up the schema change from the previous commit so docs/ matches
the provider schema and CI's docs-check passes.

Made-with: Cursor
@caballeto caballeto merged commit 27b985f into main Apr 24, 2026
6 checks passed
@caballeto caballeto deleted the feat/status-page-domain-primary branch April 24, 2026 08:47
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