diff --git a/vcauth/user_admin.py b/vcauth/user_admin.py index e33ed8326..b55e0c314 100644 --- a/vcauth/user_admin.py +++ b/vcauth/user_admin.py @@ -1,8 +1,12 @@ +import logging + from django.contrib import messages from django.contrib.auth.admin import UserAdmin from classification.views.classification_email_view import send_summary_email_to_user +logger = logging.getLogger(__name__) + class CustomUserAdmin(UserAdmin): def __init__(self, *args, **kwargs): @@ -19,10 +23,12 @@ def email_discordance(self, request, queryset): f"Email Server Issue or Emails disabled for sending {user.username} at {user.email} an email", messages.WARNING) - except Exception as ex: - self.message_user(request, f"Error {ex} when sending {user.username} at {user.email} an email", messages.ERROR) + except Exception: + logger.exception("Failed to send summary email to %s", user.username) + self.message_user(request, f"Failed to send email to {user.username}. Check server logs.", messages.ERROR) self.message_user(request, 'Emailed %i users' % count) + email_discordance.allowed_permissions = ['change'] email_discordance.short_description = "Email weekly summary" actions = [email_discordance]