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

Merge pull request #31407 from openedx/ashultz0/chatty-goals

Browse Source 
fix: course goals email command logs more progress info
This commit is contained in:
Andrew Shultz
2022-12-06 11:27:51 -05:00
committed by GitHub
parent 20b93b8b01 08ead3ba01
commit 10aabcc709
1 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
lms/djangoapps/course_goals/management/commands/goal_reminder_email.py
View File

@@ -153,16 +153,24 @@ class Command(BaseCommand):
courses_to_exclude = CourseOverview.objects.filter(
id__in=all_goal_course_keys, end__date__lte=sunday_date
).values_list('id', flat=True)
log.info(f"Processing course goals across {len(all_goal_course_keys)} courses "
+ f"excluding {len(courses_to_exclude)} ended courses")
count = 0
sent_count = 0
filtered_count = 0
course_goals = course_goals.exclude(course_key__in=courses_to_exclude).select_related('user').order_by('user')
log.info(f'Processing course goals, total goal count {len(course_goals)}')
for goal in course_goals:
# emulate a request for waffle's benefit
with emulate_http_request(site=Site.objects.get_current(), user=goal.user):
if self.handle_goal(goal, today, sunday_date, monday_date):
count += 1
sent_count += 1
else:
filtered_count += 1
if (sent_count + filtered_count) % 1000 == 0:
log.info(f'Processing course goals: sent {sent_count} filtered {filtered_count}')
log.info(f'Sent {count} emails')
log.info(f'Sent {sent_count} emails, filtered out {filtered_count} emails')
@staticmethod
def handle_goal(goal, today, sunday_date, monday_date):