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

Email functionality #8

Closed
wants to merge 4 commits into from
Closed

Email functionality #8

wants to merge 4 commits into from

Conversation

ericdc1
Copy link

@ericdc1 ericdc1 commented Sep 11, 2012

I added the send email functionality related to issue #6 I submitted yesterday. I am not certain I did enough separation for your liking but thought you may want to see what I had in mind. This is my first pull request on Github so I apologize in advance if I did something wrong.

It may be better to use smtp settings in the system.net section of web.config rather than the setting it in the section of in web.config.

@NickCraver
Copy link
Owner

Via a different approach this is now implemented (using a razor view for the email, etc.)...we can continue to improve this. of the big improvements: linking to the error has a caveat that needs addressing first (the email is sent before the real GUID to link to is known...this should be addressed soon then I can look at linking to the error).

@NickCraver NickCraver closed this Nov 28, 2012
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.

None yet

2 participants