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

refactor: remove temporary logs and fix email styling for reported content.

Browse Source
This commit is contained in:
SaadYousaf
2022-05-13 15:01:19 +05:00
committed by Saad Yousaf
parent 0edb0bb8b6
commit a26a6b83d5
3 changed files with 5 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
lms/djangoapps/discussion/signals/handlers.py
View File

@@ -124,5 +124,4 @@ def send_message(comment, site): # lint-amnesty, pylint: disable=missing-functi
def send_message_for_reported_content(user, post, site, sender): # lint-amnesty, pylint: disable=missing-function-docstring
context = create_message_context_for_reported_content(user, post, site, sender)
log.info('Reported content email notification task prepared.')
tasks.send_ace_message_for_reported_content.apply_async(args=[context], countdown=120)

2
lms/djangoapps/discussion/tasks.py
View File

@@ -93,7 +93,6 @@ def send_ace_message(context): # lint-amnesty, pylint: disable=missing-function
@shared_task(base=LoggedTask)
@set_code_owner_attribute
def send_ace_message_for_reported_content(context): # lint-amnesty, pylint: disable=missing-function-docstring
log.info('Reported content email notification task started.')
context['course_id'] = CourseKey.from_string(context['course_id'])
context['course_name'] = modulestore().get_course(context['course_id']).display_name
@@ -114,7 +113,6 @@ def send_ace_message_for_reported_content(context): # lint-amnesty, pylint: dis
log.info(f'Sending forum reported content email notification with context {message_context}')
ace.send(message)
# TODO: add tracking for reported content email
log.info('Reported content email notification task completed.')
def _track_notification_sent(message, context):

13
lms/djangoapps/discussion/templates/discussion/edx_ace/reportedcontentnotification/email/body.html
View File

@@ -15,14 +15,11 @@
{% endfilter %}
{% interpolate_html replied_to_text start_tag='<b>'|safe end_tag='</b>'|safe %}
</h2>
<div style="border-left: 1px solid rgba(0,0,0,0.25);
padding: 1px 1px 1px 15px;
margin: 20px 20px 30px 30px;
color: rgba(0,0,0,.75);">
<b> You are receiving this email because the following {{ content_type }} was reported for review </b>
<b> Title </b>
{{ comment_body }}
</div>
<h3> <b> You are receiving this email because the following {{ content_type }} was reported for review </b> </h3>
<h3> <b> Title </b> </h3>
<br />
{{ comment_body }}
{% filter force_escape %}
{% blocktrans asvar course_cta_text %}Go to Discussion{% endblocktrans %}