Skip to content

fix: Update EmailAttachmentsNode to handle optional attachments and s…#204

Merged
felix-schultz merged 1 commit intoalphafrom
dev
Aug 18, 2025
Merged

fix: Update EmailAttachmentsNode to handle optional attachments and s…#204
felix-schultz merged 1 commit intoalphafrom
dev

Conversation

@felix-schultz
Copy link
Copy Markdown
Member

This pull request updates the handling of email attachments in the EmailAttachmentsNode logic to improve type safety and robustness. The main changes focus on how the attachments are represented and processed, ensuring the code handles missing attachment data more gracefully.

Attachment handling improvements:

  • Changed the schema registration for the attachments pin to use ValueType::Array and set the schema to Attachment, instead of an optional vector of attachments. This makes the expected data type more explicit and consistent.
  • Updated the value assignment for the attachments pin to use email.attachments.unwrap_or_default(), ensuring that an empty array is used when there are no attachments, preventing potential errors from None values.

@felix-schultz felix-schultz merged commit 2dcf13a into alpha Aug 18, 2025
1 of 5 checks 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.

1 participant