Move emailing to the member as last operation since that cannot be rolled back

Author: rambo

project/examples/handlers.py

@@ -43,10 +43,6 @@ def on_approved(self, application, member):
         msg = "on_approved called for %s" % application
         logger.info(msg)
         print(msg)
-        mail = EmailMessage()
-        mail.to = [member.email, ]
-        mail.body = """Your membership has been approved, your member id is #%d""" % member.member_id
-        mail.send()
 
         # Auto-add the membership fee as recurring transaction
         membership_fee = env.float('MEMBEREXAMPLE_MEMBERSHIP_FEE', default=None)
@@ -73,6 +69,11 @@ def on_approved(self, application, member):
             mail.body = 'subscribe'
             mail.send()
 
+        mail = EmailMessage()
+        mail.to = [member.email, ]
+        mail.body = """Your membership has been approved, your member id is #%d""" % member.member_id
+        mail.send()
+
 
 class TransactionHandler(BaseTransactionHandler):