-
Notifications
You must be signed in to change notification settings - Fork 9
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
Test email options #283
base: master
Are you sure you want to change the base?
Test email options #283
Conversation
email.Body = GetBody(htmlDocument); | ||
email.Subject = GetSubject(htmlDocument, emailType); | ||
|
||
if (string.IsNullOrEmpty(smtpServer)) | ||
return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see you put the calls to GetBody()
and GetSubject()
before checking for smtpServer
. Should we also modify WriteEmailToFile()
so it can be invoked even without an SMTP server?
…edEmailDataSource model
… could fail to be loaded into the database
Pushing what I have so far. I didn't get around to actually cleaning up the |
Added endpoint to test email via download of file or writing it to the fileshare