Skip to content

Add files via upload#944

Open
maxgod1 wants to merge 12 commits intoDomain-Connect:masterfrom
maxgod1:master
Open

Add files via upload#944
maxgod1 wants to merge 12 commits intoDomain-Connect:masterfrom
maxgod1:master

Conversation

@maxgod1
Copy link
Copy Markdown

@maxgod1 maxgod1 commented Apr 5, 2026

Description

Three templates for Forgigs (https://forgigs.com), a platform that lets creators publish custom pages with newsletters and e-commerce. Users connect their own domain to their Forgigs page and can send emails (newsletters, order confirmations) from that domain.

forgigs.com.page-routing-apex — Points an apex domain (e.g. example.com) to the user's Forgigs page. Creates a single A record.

forgigs.com.page-routing-subdomain — Points a subdomain (e.g. www.example.com) to the user's Forgigs page. Creates a single CNAME record. Uses hostRequired: true so the service provider passes the subdomain label via the host parameter.

forgigs.com.email-sending — Sets up email authentication so Forgigs can send emails on behalf of the user's domain. Creates SPF (via SPFM) and DKIM records at the apex. Used independently of the routing templates so subdomain users can also configure email sending on their apex domain.

Type of change

  • New template

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

Checklist of common problems

  • syncPubKeyDomain is set
  • warnPhishing is not set alongside syncPubKeyDomain
  • syncRedirectDomain is set
  • no TXT record contains SPF content — SPFM type is used instead
  • txtConflictMatchingMode — not applicable, no unique TXT records
  • no bare variable used as full record value
  • no bare variable used as full host label — exception: DKIM host variables (%dkim1_host%, %dkim2_host%, %dkim3_host%) are bare because Resend generates per-domain DKIM selectors that cannot be predicted at template authoring time
  • no variable used in host field to create a subdomain — forgigs.com.page-routing-subdomain uses hostRequired: true with host: @; the subdomain label is passed by the service provider via the host parameter
  • %host% does not appear in any host attribute
  • essential — not applicable

Online Editor test results

Test forgigs.com/email-sending example.com/@
Test forgigs.com/email-sending example.com/www
Test forgigs.com/page-routing-apex example.com/@
Test forgigs.com/page-routing-apex example.com/www
Test forgigs.com/page-routing-subdomain example.com/www

@github-actions github-actions bot added Test links missing No test links from Online Editor provided warnPhishing warnPhishing is set. Template is not secured. labels Apr 5, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

JSON Filename Check Passed

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

JSON Schema Validation Passed

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

Linter OK

forgigs.com.email-sending.json

Level Code Note

forgigs.com.page-routing-apex.json

Level Code Note

forgigs.com.page-routing-subdomain.json

Level Code Note

maxgod1 added 2 commits April 5, 2026 11:10
Updated variable descriptions and changed TXT record to SPFM.
@github-actions github-actions bot removed the warnPhishing warnPhishing is set. Template is not secured. label Apr 5, 2026
@github-actions github-actions bot added the hostRequired hostRequired is set label Apr 5, 2026
@github-actions github-actions bot removed the hostRequired hostRequired is set label Apr 5, 2026
@github-actions github-actions bot added Test links missing No test links from Online Editor provided hostRequired hostRequired is set and removed Test links missing No test links from Online Editor provided labels Apr 5, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

PR Description Check Passed

All required sections are filled in correctly.

Details
  OK  Type of change: 1/1 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: 5 link(s) found
  OK  Template coverage: all 3 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

},
{
"type": "CNAME",
"host": "%dkim1_host%",
Copy link
Copy Markdown
Member

@pawel-kow pawel-kow Apr 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be done better. See Rule 7 in README.

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

Labels

hostRequired hostRequired is set

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants