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

fix(connector): fix missuse of replyTo field #7062

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

simeng-li
Copy link
Contributor

Summary

fix the misuse of the replyTo field.

For some email providers, this field can indicate the reply email address. Previously, it was confused with the alias of send to email address.

Testing

Unit test updated

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

fix the miss use of replyTo field. Should not be used as the alias of to email
Copy link

COMPARE TO master

Total Size Diff 📉 -246 Bytes

Diff by File
Name Diff
packages/connectors/connector-mailgun/src/index.test.ts 📉 -84 Bytes
packages/connectors/connector-mailgun/src/index.ts 📉 -40 Bytes
packages/connectors/connector-sendgrid-email/src/index.test.ts 📉 -72 Bytes
packages/connectors/connector-sendgrid-email/src/index.ts 📉 -119 Bytes
packages/connectors/connector-smtp/src/index.test.ts 📉 -73 Bytes
packages/toolkit/connector-kit/src/types/email-template.ts 📉 -172 Bytes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant