Patch #216

Email settings

Added by Ted Wise almost 13 years ago. Updated over 9 years ago.

Status:ClosedStart date:
Priority:NormalDue 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).

redmine-email-settings.patch Magnifier (16.3 KB) Ted Wise, 2008-02-03 11:00

History

#1 Updated by Jean-Philippe Lang almost 13 years ago

No patch attached.

#2 Updated by Rocco Stanzione almost 12 years ago

I see a patch!

#3 Updated by Jean-Philippe Lang almost 12 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:

  1. It won't play well with plugins that introduce new email delivery settings (eg. ActionMailer TLS)
  2. 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 9 years ago

  • Status changed from New to Closed

Closing as it won't be applied.

Also available in: Atom PDF