Skip to content

Commit

Permalink
send mail only if SMTP was configured
Browse files Browse the repository at this point in the history
  • Loading branch information
mrsaicharan1 committed Jul 23, 2019
1 parent 7b54d2a commit 3780030
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions app/api/helpers/tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
from app.api.helpers.utilities import strip_tags
from app.models.session import Session
from app.models.speaker import Speaker

from app.api.helpers.mail import check_smtp_config

"""
Define all API v2 celery tasks here
Expand Down Expand Up @@ -183,15 +183,21 @@ def export_event_task(self, email, event_id, settings):
result = {
'download_url': download_url
}

logging.info('Exporting done.. sending email')
send_export_mail(email=email, event_name=event.name, download_url=download_url)
if check_smtp_config:
send_export_mail(email=email, event_name=event.name, download_url=download_url)
else:
logging.warning('Error in sending export success mail')
send_notif_after_export(user=user, event_name=event.name, download_url=download_url)
except Exception as e:
result = {'__error': True, 'result': str(e)}
logging.warning('Error in exporting.. sending email')
send_export_mail(email=email, event_name=event.name, error_text=str(e))
if check_smtp_config:
send_export_mail(email=email, event_name=event.name, error_text=str(e))
else:
logging.warning('Error in sending export error mail')
send_notif_after_export(user=user, event_name=event.name, error_text=str(e))

return result


Expand Down

0 comments on commit 3780030

Please sign in to comment.