Skip to content

chore(kno-11112): reference payload size limitation in attachment docs#1361

Merged
scotidodson merged 3 commits intomainfrom
sd-KNO-11112-attachment-payload-limit
Mar 23, 2026
Merged

chore(kno-11112): reference payload size limitation in attachment docs#1361
scotidodson merged 3 commits intomainfrom
sd-KNO-11112-attachment-payload-limit

Conversation

@scotidodson
Copy link
Contributor

KNO-11112: Docs — update email attachment docs to reference the total payload limit

Summary

  • Expands the content property description in the attachment object table to clarify that the 10MB limit applies to the full trigger payload (not per attachment), links to the trigger payload docs, and notes that email providers may enforce stricter limits.
  • General cleanup: removed emoji from intro steps, consolidated two **Note** callouts into prose, folded multiple-attachments mention into section intro.

Changes

  • content/integrations/email/attachments.mdxcontent table row updated with payload limit link and provider limit note; prose and style cleanup throughout

Notes

The 10MB payload limit is documented at send-notifications/triggering-workflows/api.mdx line 64. No changes needed there.

@linear
Copy link

linear bot commented Mar 23, 2026

@vercel
Copy link

vercel bot commented Mar 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 23, 2026 10:14pm

Request Review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Risk LOW: Clarifies the 10MB trigger payload limit in email attachment docs and performs minor prose cleanup.

Reasons

  • Only a single .mdx content file is modified (content/integrations/email/attachments.mdx)
  • Small diff: 6 additions and 10 deletions (16 total lines changed)
  • No new pages, sidebar changes, component changes, or dependency updates
  • Changes are editorial: consolidating notes, updating a table description, and adding a cross-reference link
  • Author (scotidodson) is a known repository contributor

Notes

  • Verify the cross-reference link to /send-notifications/triggering-workflows/api#trigger-payload resolves correctly on the preview deployment
  • The Callout component change (from type="info" + title="Note:" to emoji="💡" + title="Resend inline images.") is a minor style update — confirm it renders as expected
Open in Web View Automation 

Sent by Cursor Automation: Docs PR classifier

@scotidodson scotidodson merged commit 642a4f2 into main Mar 23, 2026
5 checks passed
@scotidodson scotidodson deleted the sd-KNO-11112-attachment-payload-limit branch March 23, 2026 22:16
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