From 59dd149137e88ac8051cb58a9b2aaff4dd5d4d9e Mon Sep 17 00:00:00 2001 From: Matthew Piatetsky Date: Fri, 3 Dec 2021 10:51:18 -0500 Subject: [PATCH] fix: use the same logo in ace emails as we use in our braze emails (#29494) --- lms/djangoapps/branding/api.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lms/djangoapps/branding/api.py b/lms/djangoapps/branding/api.py index 5f4f06d9d2..91ba26ba12 100644 --- a/lms/djangoapps/branding/api.py +++ b/lms/djangoapps/branding/api.py @@ -651,4 +651,6 @@ def get_logo_url_for_email(): Returns the url for the branded logo image for embedding in email templates. """ default_logo_url = getattr(settings, 'DEFAULT_EMAIL_LOGO_URL', None) - return getattr(settings, 'LOGO_URL_PNG', None) or default_logo_url + # The LOGO_URL_PNG might be reused in the future for other things, so including an email specific png logo + return (getattr(settings, 'LOGO_URL_PNG_FOR_EMAIL', None) or + getattr(settings, 'LOGO_URL_PNG', None) or default_logo_url)