Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use the same logo in ace emails as we use in our braze emails (#29494)

Browse Source
This commit is contained in:
Matthew Piatetsky
2021-12-03 10:51:18 -05:00
committed by GitHub
parent a1c34aa7cd
commit 59dd149137
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lms/djangoapps/branding/api.py
View File

@@ -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)