Skip to content

source-sendgrid: patch templates to remove updated_at datetime format#353

Merged
Alex-Bair merged 1 commit into
masterfrom
bair/source-sendgrid-remove-datetime-format-from-templates-updated-at
Feb 25, 2026
Merged

source-sendgrid: patch templates to remove updated_at datetime format#353
Alex-Bair merged 1 commit into
masterfrom
bair/source-sendgrid-remove-datetime-format-from-templates-updated-at

Conversation

@Alex-Bair
Copy link
Copy Markdown
Member

@Alex-Bair Alex-Bair commented Feb 24, 2026

The updated_at field for templates are not ISO 8601 datetimes since they use spaces to separate the date and time parts. (API docs reference). We need to patch out the datetime format to avoid schema violation errors when capturing templates documents.

Confirmed with flowctl raw discover that format: datetime is no longer present in the discovered schema for the top level updated_at for the templates resource.

…rmat

The `updated_at` field for templates are not RFC3339 datetimes since they use spaces to separate the date and time parts. ([API docs reference](https://www.twilio.com/docs/sendgrid/api-reference/transactional-templates/retrieve-paged-transactional-templates#responses)). We need to patch out the datetime format to avoid schema violation errors when capturing `templates` documents.
@Alex-Bair Alex-Bair requested a review from a team February 24, 2026 18:45
Copy link
Copy Markdown

@nicolaslazo nicolaslazo left a comment

Choose a reason for hiding this comment

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

👍

@Alex-Bair Alex-Bair merged commit 36a7da8 into master Feb 25, 2026
45 of 57 checks passed
@Alex-Bair Alex-Bair deleted the bair/source-sendgrid-remove-datetime-format-from-templates-updated-at branch February 25, 2026 13:07
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.

2 participants