Skip to content

add MonCoach, MonCoach — site en marque blanche - #1535

Open
akambaki wants to merge 1 commit into
Domain-Connect:masterfrom
akambaki:add-moncoach-site
Open

add MonCoach, MonCoach — site en marque blanche#1535
akambaki wants to merge 1 commit into
Domain-Connect:masterfrom
akambaki:add-moncoach-site

Conversation

@akambaki

@akambaki akambaki commented Aug 6, 2026

Copy link
Copy Markdown

Description

New template for MonCoach (moncoach.io), a French white-label coaching platform. The site service connects a customer's own domain to their MonCoach site: apex A record to the platform edge, wildcard CNAME for the app/booking subdomains, and an ownership-verification TXT record. Template variables (ip, token) are generated server-side by the platform when building the apply URL.

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 — no logoUrl in this template

Also validated with dc-template-linter -loglevel error -tolerate info (exit 0; only infos: DCTL1029 no syncPubKeyDomain — justified below — and DCTL1021 _moncoach-verify not an IANA label, expected for a provider-specific verification record).

Checklist of common problems

  • syncPubKeyDomain is set — justification: apply URLs are generated exclusively server-side by our platform (no user-editable parameters), and warnPhishing: true is set so registrars display the confirmation warning. We plan to add syncPubKeyDomain + signed apply URLs (key published at _dck1.moncoach.io) in a version 2 once our signing endpoint ships.
  • warnPhishing is not set alongside syncPubKeyDomain — only warnPhishing is present
  • syncRedirectDomain is set whenever the template uses redirect_uri in the synchronous flow (syncRedirectDomain: "moncoach.io")
  • no TXT record contains SPF content
  • txtConflictMatchingMode is set on every TXT record that must be unique (All on the _moncoach-verify token record)
  • no variable is used as a bare full record value — justification: the apex A record value is %ip%, the address of the platform edge. An IP cannot carry a fixed prefix; the value is not user-supplied (our backend injects the current edge IP when building the apply URL). Kept as a variable rather than hard-coded so an edge re-IP does not require a registry-wide template resubmission.
  • no bare variable is used as the full host label — all host fields are fixed (@, *, _moncoach-verify)
  • no variable is used in the host field to create a subdomain
  • %host% does not appear explicitly in any host attribute
  • essential is set on records the end user may need to modify independently — not applicable: all three records are required for the service to function

Online Editor test results

Editor test link(s):

Both tests applied the expected three records (apex A → edge IP, wildcard CNAME → apex, _moncoach-verify TXT → token).

@github-actions github-actions Bot added the no syncPubKeyDomain syncPubKeyDomain not set. Template is not secured. label Aug 6, 2026
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

JSON Filename Check Passed

@github-actions github-actions Bot added PR description incomplete The PR description template was not filled in at all, altered or filled in improperly. warnPhishing warnPhishing is set. Template is not secured. labels Aug 6, 2026
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

JSON Schema Validation Passed

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

PR Description Check Failed

The PR description is missing required elements. Please update it according to the PR template.

Details
  OK  Type of change: 1/4 checkboxes ticked
  OK  Checklist of common problems: 8/10 checkboxes ticked
  OK  Online Editor test results: 2 link(s) found
  OK  Template coverage: all 1 template(s) covered

Labels to add: PR description incomplete
Labels to remove: Checklist of common problems not complete, Forged editor links, Test links missing

PR description check FAILED:
  FAIL  'How Has This Been Tested?': 1/3 checkboxes are not ticked

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Linter OK

moncoach.io.site.json

Level Code Note
info DCTL1029 template does not have syncPubKeyDomain
info DCTL1021 missing from iana definitions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no syncPubKeyDomain syncPubKeyDomain not set. Template is not secured. PR description incomplete The PR description template was not filled in at all, altered or filled in improperly. warnPhishing warnPhishing is set. Template is not secured.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant