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 DKIM support, fix bcc and cc, and add markdown support #17

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

kcoderhtml
Copy link

@kcoderhtml kcoderhtml commented Apr 27, 2024

I added DKIM support and made the error logging more effective by logging the error returned by mailchannels. BTW, this is a lifesaver for saving on time and effort writing my own worker; really appreciate all the hard work!

Closes: #8

@kcoderhtml
Copy link
Author

I also fixed bcc and cc as they needed to be in the personalization object as shown here: https://api.mailchannels.net/tx/v1/documentation

@kcoderhtml kcoderhtml changed the title feat: add DKIM support feat: add DKIM support and fix bcc and cc Apr 27, 2024
@kcoderhtml
Copy link
Author

Just finished adding markdown support as well

@kcoderhtml kcoderhtml changed the title feat: add DKIM support and fix bcc and cc feat: add DKIM support, fix bcc and cc, and add markdown support Apr 28, 2024
@agusmdev
Copy link

Nice work @kcoderhtml ! I tried your code and it worked perfectly, I was able to set DKIM properly

@kcoderhtml
Copy link
Author

I'm glad it worked for you!

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.

DKIM Support
2 participants