Skip to content

Add enwella.com clinic-website template#1449

Open
vshaveyko wants to merge 5 commits into
Domain-Connect:masterfrom
vshaveyko:add-enwella-clinic-website
Open

Add enwella.com clinic-website template#1449
vshaveyko wants to merge 5 commits into
Domain-Connect:masterfrom
vshaveyko:add-enwella-clinic-website

Conversation

@vshaveyko

@vshaveyko vshaveyko commented Jul 22, 2026

Copy link
Copy Markdown

Description

Domain Connect service-provider template for one-click custom-domain setup of an Enwella-hosted clinic website — provider enwella.com, service clinic-website. It applies an apex A record to the Enwella ingress, a www CNAME to the tenant subdomain (%slug%.enwella.com, which resolves via the *.enwella.com wildcard so it survives ingress IP changes), and a domain-ownership TXT. Synchronous apply requests are signed — syncPubKeyDomain: enwella.com, with the RSA public key published at _dck1.enwella.com (key=_dck1).

Type of change

  • New template
  • Bug fix (non-breaking change which fixes an issue in the template)
  • New feature (non-breaking change which adds functionality to the template)
  • Breaking change (fix or feature that would cause existing template behavior to be not backward compatible)

How Has This Been Tested?

  • Template functionality checked using Online Editor
  • Template file name follows the pattern <providerId>.<serviceId>.json
  • resource URL provided with logoUrl is actually served by a webserver (https://enwella.com/icon.png)

Checklist of common problems

  • syncPubKeyDomain is set — enwella.com; public key live at _dck1.enwella.com
  • warnPhishing is not set alongside syncPubKeyDomain
  • syncRedirectDomain is set whenever the template uses redirect_uri — N/A, this template does not use redirect_uri
  • no TXT record contains SPF content — the only TXT is a domain-ownership token
  • txtConflictMatchingMode is set on TXT records that must be unique — N/A, no DMARC / unique-prefix TXT
  • no variable is used as a bare full record value — the _enwella-verify TXT uses a bare %token%. This is a domain-ownership verification token, so a bare value is necessary and intended (same pattern as google.com.domain-verification's bare %verifytxt%).
  • no bare variable is used as the full host label — all hosts are fixed literals (_enwella-verify, www, @)
  • no variable is used in the host field to create a subdomain
  • %host% does not appear explicitly in any host attribute
  • essential is set to OnApply on records the end user may need to modify — N/A, no DMARC / user-editable records

Online Editor test results

Editor test link(s):

Domain Connect template for one-click DNS setup of an Enwella-hosted
clinic website: apex A record, www CNAME to the tenant subdomain, and a
domain-ownership TXT record. Signed apply requests via syncPubKeyDomain
enwella.com.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vshaveyko
vshaveyko marked this pull request as ready for review July 22, 2026 15:18
@github-actions

Copy link
Copy Markdown

JSON Filename Check Passed

@github-actions

Copy link
Copy Markdown

JSON Schema Validation Passed

@github-actions

Copy link
Copy Markdown

Linter OK

enwella.com.clinic-website.json

Level Code Note
info DCTL1021 missing from iana definitions

@github-actions github-actions Bot added the Test links missing No test links from Online Editor provided label Jul 24, 2026
@github-actions github-actions Bot removed the Test links missing No test links from Online Editor provided label Jul 24, 2026
@github-actions

Copy link
Copy Markdown

PR Description Check Passed

All required sections are filled in correctly.

Details
  OK  Type of change: 1/4 checkboxes ticked
  OK  How Has This Been Tested?: all 3 checkboxes ticked
  OK  Checklist of common problems: 10/10 checkboxes ticked
  OK  Online Editor test results: 2 link(s) found
  OK  Template coverage: all 1 template(s) covered
Labels to remove: Checklist of common problems not complete, Forged editor links, PR description incomplete, Test links missing

PR description check PASSED

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.

2 participants