Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add support for mail_settings message field #118

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

spencewenski
Copy link
Contributor

Adds support for the mail_settings field of the mail-send API. This field includes the sandbox_mode setting, which allows sending the API to Sendgrid without actually sending the email to the recipient(s), which is useful for testing.

Closes #103

Adds support for the `mail_settings` field of the
[mail-send](https://www.twilio.com/docs/sendgrid/api-reference/mail-send/mail-send#request-body)
API. This field includes the `sandbox_mode` setting, which allows
sending the API to Sendgrid without actually sending the email to the
recipient(s), which is useful for testing.

Closes gsquire#103
Copy link
Owner

@gsquire gsquire left a comment

Choose a reason for hiding this comment

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

Thanks for adding this in! I just have a couple minor comments.

src/v3/message.rs Show resolved Hide resolved
src/v3/message.rs Show resolved Hide resolved
When the `bypass_list_management` field is present, the other bypass
filters (`bypass_spam_management`, `bypass_bounce_management`, and
`bypass_unsubscribe_management`) are ignored. Model this as an enum in
the code.
src/v3/message.rs Show resolved Hide resolved
@gsquire gsquire merged commit 40bacd1 into gsquire:master Oct 10, 2024
4 checks passed
@gsquire
Copy link
Owner

gsquire commented Oct 10, 2024

Awesome, thanks for this patch. I will push a new release after I do some housekeeping.

gsquire added a commit that referenced this pull request Oct 10, 2024
This incorporates #118 for a new release.
@gsquire gsquire mentioned this pull request Oct 10, 2024
gsquire added a commit that referenced this pull request Oct 10, 2024
This incorporates #118 for a new release.
@gsquire
Copy link
Owner

gsquire commented Oct 10, 2024

Alright, version 0.23.0 has these changes.

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.

Sandbox mode
2 participants