Patch #216
Email settings
Status: | Closed | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 0% | |
Category: | Email notifications | |||
Target version: | - |
Description
Creates an admin interface for the email settings. Uses the existing Settings table to store the new settings. Adds
a check to the info page as to whether email has been configured. Disables emails by default. Needs additional translations
for newly added strings and an additional icon for the admin page (using the existing mail notifications icon now).
History
#1
Updated by Jean-Philippe Lang almost 14 years ago
No patch attached.
#2
Updated by Rocco Stanzione almost 13 years ago
I see a patch!
#3
Updated by Jean-Philippe Lang almost 13 years ago
- Category set to Email notifications
Yes, the patch was added later.
Web based SMTP configuration could be convenient for some people but here are the main reasons why I don't apply this patch:
- It won't play well with plugins that introduce new email delivery settings (eg. ActionMailer TLS)
- It won't work as expected when running multiple instances of the app (eg. with a mongrel cluster)
#4
Updated by Jean-Baptiste Barth over 10 years ago
- Status changed from New to Closed
Closing as it won't be applied.