seq-app-htmlemail icon indicating copy to clipboard operation
seq-app-htmlemail copied to clipboard

Email+ - Add optional CC and BCC fields

Open pillad3 opened this issue 5 years ago • 7 comments

Add an optional setting to Email+ instances to set one or more CC or BCC addresses to emails with the standard Handlebars support. We currently achieve a similar effect by creating a duplicate Email+ instance with a different set of addresses, but this quickly becomes unmanageable.

pillad3 avatar Apr 02 '20 01:04 pillad3

Hi! Thanks for the note. To make sure everything lines up, can you please let us know what the source data that you would be templating over would look like? I.e. how would the addresses be formatted based on the log data? Thanks!

nblumhardt avatar Apr 02 '20 22:04 nblumhardt

Thanks for the quick reply! I was thinking just having a comma-separated field of email addresses, exactly the same as the "To address" field. Addresses that come from log data could use standard Handlebars syntax to input log event properties directly without any formatting.

Do you have an example of what you mean by formatting addresses based on log data? Maybe I'm not understanding the question properly.

pillad3 avatar Apr 06 '20 13:04 pillad3

Hi! Thanks for your reply - sorry, I was mistaken in thinking the current TO box didn't support templates, I see that it now does, so please ignore my question :-)

We'll give this one some thought; thanks 👍

nblumhardt avatar Apr 07 '20 21:04 nblumhardt

I've published https://www.nuget.org/packages/Seq.App.EmailPlus-Enhanced/ to Nuget as a test package.

This has the code from my two open pull requests, which allows for testing of the enhanced code.

This includes the option for CC, BCC, and ReplyTo fields. Handlebars syntax is supported as per To address.

MattMofDoom avatar Feb 20 '22 23:02 MattMofDoom

That's great - thanks for the follow-up, @MattMofDoom :+1:

nblumhardt avatar Feb 21 '22 01:02 nblumhardt

@nblumhardt No problem. I actually used it to fix a prod problem for me that really needed the ReplyTo and CC fields in particular.

MattMofDoom avatar Feb 21 '22 02:02 MattMofDoom

Thanks, @MattMofDoom!

pillad3 avatar Feb 22 '22 16:02 pillad3