Adding issue date to CertificateBase (#3025)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: CP <[email protected]>

Author: 3 people

cms/models.py

@@ -4,7 +4,7 @@
 import logging
 import re
 import uuid
-from datetime import datetime, timedelta
+from datetime import datetime
 from json import dumps
 from urllib.parse import quote_plus
 
@@ -459,16 +459,7 @@ def get_context(self, request, *args, **kwargs):
                 "uuid": "fake-uuid",
                 "learner_name": "Anthony M. Stark",
                 "product_name": product_name,
-                "start_date": (
-                    self.parent.product.first_unexpired_run.start_date
-                    if self.parent.product.first_unexpired_run
-                    else datetime.now()  # noqa: DTZ005
-                ),
-                "end_date": (
-                    self.parent.product.first_unexpired_run.end_date
-                    if self.parent.product.first_unexpired_run
-                    else datetime.now() + timedelta(days=45)  # noqa: DTZ005
-                ),
+                "issue_date": self.issue_date,
                 "CEUs": self.CEUs,
                 "is_program_certificate": is_program_certificate,
             }

cms/templates/certificate_page.html

@@ -73,7 +73,7 @@ <h2>Congratulations, {{ learner_name }}!</h2>
                     {% endif %}
                     <span class="award-text">
                         {% if page.CEUs %} Awarded {{ CEUs }} Continuing Education Units (CEUs) <br> {% endif %}
-                        {{ start_date|date }} - {{ end_date|date }}
+                        Issued: {{ end_date|date }}
                     </span>
                     <div class="row justify-content-center certify-by-row">
                         {% for signatory in page.signatory_pages %}

courses/migrations/0020_courseruncertificate_certificate_page_revision.py

@@ -8,10 +8,11 @@ def set_current_certificate_revision(apps, schema_editor):
     """
     Fetch all CourseRunCertificates and update certificate_page_revision to latest revision
     """
-    from cms.models import CertificatePage
-    from courses.models import CourseRunCertificate
-
-    course_run_certificates = CourseRunCertificate.objects.all()
+    CourseRunCertificate = apps.get_model("courses", "CourseRunCertificate")
+    CertificatePage = apps.get_model("cms", "CertificatePage")
+    course_run_certificates = CourseRunCertificate.objects.only(
+        "id", "course_run", "certificate_page_revision"
+    ).all()
     for cert in course_run_certificates:
         certificate_page = (
             cert.course_run.course.page.get_children()
@@ -34,6 +35,10 @@ class Migration(migrations.Migration):
     dependencies = [
         ("wagtailcore", "0070_rename_pagerevision_revision"),
         ("courses", "0019_add_is_self_paced_course_run"),
+        (
+            "cms",
+            "0023_certificateindexpage_certificatepage_signatoryindexpage_signatorypage",
+        ),
     ]
 
     operations = [

courses/migrations/0073_courseruncertificate_issue_date_and_more.py

@@ -0,0 +1,22 @@
+# Generated by Django 4.2.25 on 2025-10-21 14:55
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("courses", "0072_alter_programcollectionitem_sort_order"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="courseruncertificate",
+            name="issue_date",
+            field=models.DateTimeField(blank=True, db_index=True, null=True),
+        ),
+        migrations.AddField(
+            model_name="programcertificate",
+            name="issue_date",
+            field=models.DateTimeField(blank=True, db_index=True, null=True),
+        ),
+    ]

courses/migrations/0074_populate_issue_date.py

@@ -0,0 +1,29 @@
+# Generated by Django 4.2.25 on 2025-10-21 14:59
+
+from django.db import migrations
+
+
+def populate_issue_date(apps, schema):
+    CourseRunCertificate = apps.get_model("courses", "CourseRunCertificate")
+    for certificate in CourseRunCertificate.objects.all():
+        if certificate.issue_date is None:
+            certificate.issue_date = certificate.created_on
+            certificate.save(update_fields=["issue_date"])
+
+    ProgramCertificate = apps.get_model("courses", "ProgramCertificate")
+    for certificate in ProgramCertificate.objects.all():
+        if certificate.issue_date is None:
+            certificate.issue_date = certificate.created_on
+            certificate.save(update_fields=["issue_date"])
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("courses", "0073_courseruncertificate_issue_date_and_more"),
+    ]
+
+    operations = [
+        migrations.RunPython(
+            populate_issue_date, reverse_code=migrations.RunPython.noop
+        ),
+    ]

courses/models.py

@@ -1286,6 +1286,7 @@ class BaseCertificate(models.Model):
         help_text="Indicates whether or not the certificate is revoked",
         verbose_name="revoked",
     )
+    issue_date = models.DateTimeField(null=True, blank=True, db_index=True)
 
     class Meta:
         abstract = True