diff --git a/lms/djangoapps/course_goals/management/commands/goal_reminder_email.py b/lms/djangoapps/course_goals/management/commands/goal_reminder_email.py index 032eda2f36..35b3cde253 100644 --- a/lms/djangoapps/course_goals/management/commands/goal_reminder_email.py +++ b/lms/djangoapps/course_goals/management/commands/goal_reminder_email.py @@ -7,7 +7,6 @@ import logging from django.conf import settings from django.contrib.sites.models import Site from django.core.management.base import BaseCommand -from django.urls import reverse from edx_ace import ace from edx_ace.message import Message from edx_ace.recipient import Recipient @@ -53,10 +52,7 @@ def send_ace_message(goal): course_home_url = get_learning_mfe_home_url(course_key=goal.course_key, view_name='home') - goals_unsubscribe_url = settings.LEARNING_MICROFRONTEND_URL + reverse( - 'course-home:unsubscribe-from-course-goal', - kwargs={'token': goal.unsubscribe_token} - ) + goals_unsubscribe_url = f'{settings.LEARNING_MICROFRONTEND_URL}/goal-unsubscribe/{goal.unsubscribe_token}' language = get_user_preference(user, LANGUAGE_KEY) diff --git a/lms/djangoapps/course_goals/templates/course_goals/edx_ace/goalreminder/email/body.html b/lms/djangoapps/course_goals/templates/course_goals/edx_ace/goalreminder/email/body.html index da1124fb01..966be4aff1 100644 --- a/lms/djangoapps/course_goals/templates/course_goals/edx_ace/goalreminder/email/body.html +++ b/lms/djangoapps/course_goals/templates/course_goals/edx_ace/goalreminder/email/body.html @@ -2,12 +2,27 @@ {% load i18n %} {% load django_markup %} {% load static %} +{% block table_style %} + style=" + font-family: Inter, Arial, sans-serif !important; + font-size: 1em; + line-height: 1.5; + max-width: 600px; + " +{% endblock %} +{% block main_style %} + style=" + padding-bottom: 30px; + box-shadow: 0 1px 5px rgba(0,0,0,0.25); + " +{% endblock %} {% block content %} {# email client support for style sheets is pretty spotty, so we have to inline all of these styles #} {# we're using important below to override inline styles and my understanding is for email clients where media queries do not work, they'll simply see the desktop css on their phone #} - - -
+ + + +
{% include "goal_reminder_banner.html" %} - diff --git a/lms/templates/goal_reminder_banner.html b/lms/templates/goal_reminder_banner.html index 3b07c5605c..5a7cff6a28 100644 --- a/lms/templates/goal_reminder_banner.html +++ b/lms/templates/goal_reminder_banner.html @@ -1,18 +1,9 @@ {# email client support for style sheets is pretty spotty, so we have to inline all of these styles #} - {% if image_url %}
diff --git a/openedx/core/djangoapps/ace_common/templates/ace_common/edx_ace/common/base_body.html b/openedx/core/djangoapps/ace_common/templates/ace_common/edx_ace/common/base_body.html index f72dd3aee1..5b056eaf13 100644 --- a/openedx/core/djangoapps/ace_common/templates/ace_common/edx_ace/common/base_body.html +++ b/openedx/core/djangoapps/ace_common/templates/ace_common/edx_ace/common/base_body.html @@ -33,6 +33,7 @@ margin: 0; padding: 0; min-width: 100%; + background-color: #f5f5f5; "> - +