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

[fix] Fix certificate available date sync (#28275)

Browse Source 
* [fix] Fix certificate available date sync

We were syncing the course available date to every course in
credentials. Since credentials doesn't understand "self-paced" courses,
or course end behaviors, some certificates were time gated incorrectly.
This check make sure to check if the course is not self-paced, and has a
CDB of 'end' before syncing the CA date.
This commit is contained in:
Thomas Tracy
2021-07-26 14:30:06 -04:00
committed by GitHub
parent 139e7ba48b
commit 666f1dadb8
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
openedx/core/djangoapps/programs/tasks.py
View File

@@ -710,11 +710,15 @@ def update_certificate_visible_date_on_course_update(self, course_key, certifica
f"Failed to update certificate availability date for course {course_key}. Reason: {error_msg}"
)
raise self.retry(exc=exception, countdown=countdown, max_retries=MAX_RETRIES)
# Always update the course certificate with the new certificate available date
update_credentials_course_certificate_configuration_available_date.delay(
str(course_key),
certificate_available_date
)
# update the course certificate with the new certificate available date if:
# - The course is self paced
# - The certificates_display_behavior is not "end"
course_overview = CourseOverview.get_from_id(course_key)
if course_overview.self_paced is False and course_overview.certificates_display_behavior == 'end':
update_credentials_course_certificate_configuration_available_date.delay(
str(course_key),
certificate_available_date
)
users_with_certificates_in_course = GeneratedCertificate.eligible_available_certificates.filter(
course_id=course_key
).values_list('user__username', flat=True)