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

Merge pull request #31410 from openedx/ashultz0/less-chatty-goals

Browse Source 
fix: intial chatty goals is a bit too chatty
This commit is contained in:
Andrew Shultz
2022-12-07 10:33:45 -05:00
committed by GitHub
parent 9c279444df 0de55deff5
commit b87c6405ff
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -159,7 +159,8 @@ class Command(BaseCommand):
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)}')
total_goals = len(course_goals)
log.info(f'Processing course goals, total goal count {total_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):
@@ -167,10 +168,10 @@ class Command(BaseCommand):
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}')
if (sent_count + filtered_count) % 10000 == 0:
log.info(f'Processing course goals: sent {sent_count} filtered {filtered_count} out of {total_goals}')
log.info(f'Sent {sent_count} emails, filtered out {filtered_count} emails')
log.info(f'Processing course goals complete: sent {sent_count} emails, filtered out {filtered_count} emails')
@staticmethod
def handle_goal(goal, today, sunday_date, monday_date):