Ashish has proposed merging 
lp:~coolyashish/postorius/add_email_address_in_settings into lp:postorius.

Requested reviews:
  Mailman Coders (mailman-coders)

For more details, see:
https://code.launchpad.net/~coolyashish/postorius/add_email_address_in_settings/+merge/252939

Previously we are getting error on add email address page in user's settings as 
there is not email_confirmation_from set in the send_confirmation_link function 
in AddressConfirmationProfile model. This change adds the required email in the 
function.
-- 
Your team Mailman Coders is requested to review the proposed merge of 
lp:~coolyashish/postorius/add_email_address_in_settings into lp:postorius.
=== modified file 'src/postorius/models.py'
--- src/postorius/models.py	2015-02-09 14:35:44 +0000
+++ src/postorius/models.py	2015-03-13 18:25:29 +0000
@@ -291,5 +291,5 @@
             settings, 'EMAIL_CONFIRMATION_SUBJECT', u'Confirmation needed')
         send_mail(email_subject,
                   get_template(template_path).render(template_context),
-                  getattr(settings, 'EMAIL_CONFIRMATION_FROM'),
+                  getattr(settings, 'EMAIL_CONFIRMATION_FROM', self.user.email),
                   [self.email])

_______________________________________________
Mailman-coders mailing list
Mailman-coders@python.org
https://mail.python.org/mailman/listinfo/mailman-coders

Reply via email to