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

Envelope enhancements #83

Open
wants to merge 51 commits into
base: dev
Choose a base branch
from

Conversation

MattMofDoom
Copy link
Contributor

Hi @nblumhardt,

PR that's intended to follow #81. I figured it would be okay to queue it up for when you get to it 😊

Feature enhancements:

  • Add Priority, with optional priority property mapping and default
  • CC, BCC, and ReplyTo added as optional envelope options with handlebars template
  • Added optional plain text alternate body template
  • Enhance email logging

Notes:

…esharper code issues, add test case for envelope template parsing
@MattMofDoom
Copy link
Contributor Author

Incorporates the merge of #86 with #81 while preserving the envelope enhancements from this PR, making this a full featured email application.

@MattMofDoom MattMofDoom mentioned this pull request Aug 31, 2021
@MattMofDoom
Copy link
Contributor Author

Hi @liammclennan and @nblumhardt,

As with #81, merging the changes from #93. This sustains the use of SmtpOptions.GetSocketOptions while retaining the added envelope functionality.

Cheers,

Matt

@MattMofDoom
Copy link
Contributor Author

Updated dependencies

@MattMofDoom
Copy link
Contributor Author

App comprising the enhancements from #81 and #83 is currently published to Nuget as https://www.nuget.org/packages/Seq.App.EmailPlus-Enhanced while this is pending review/merge.

This is working well in a production environment. More information on the features was published at https://mattmofdoom.com/seqappemailplus-enhanced-available/

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.

Email+ - Add optional CC and BCC fields
1 participant