Skip to content

docs: fix multi-channel template example missing required from fields#689

Closed
Copilot wants to merge 2 commits intoupdate-docsfrom
copilot/sub-pr-688
Closed

docs: fix multi-channel template example missing required from fields#689
Copilot wants to merge 2 commits intoupdate-docsfrom
copilot/sub-pr-688

Conversation

Copy link

Copilot AI commented Feb 25, 2026

The multi-channel messaging example omitted from on both EMAIL and TWILIO entries, making the snippet invalid against EmailTemplateType and SMSTemplateType (both require from: string).

Changes

  • docs/docs/platform-configuration/messaging.md: Added from to the EMAIL and TWILIO inputs in the multi-channel example; added inline comment on TWILIO noting from can be omitted when TWILIO_SMS_FROM env var is set (the adapter's actual fallback behavior)
return [
  {
    type: 'EMAIL',
    input: {
      from: 'shop@example.com',  // required
      to: 'admin@example.com',
      subject: `New order #${order.orderNumber}`,
      text: `Order total: ${order.pricing?.total}`,
    },
  },
  {
    type: 'TWILIO',
    input: {
      from: '+15550001234', // Omit if TWILIO_SMS_FROM env var is set
      to: '+41791234567',
      text: `New order #${order.orderNumber}`,
    },
  },
];

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Co-authored-by: fliptation <600765+fliptation@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on AI integration docs section docs: fix multi-channel template example missing required from fields Feb 25, 2026
Copilot AI requested a review from fliptation February 25, 2026 01:20
@fliptation fliptation closed this Feb 25, 2026
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.

3 participants