[OLD/ABANDONED] fix: add should_display_status_to_user method and status_changed field to VerificationAttempt model

lms/djangoapps/verify_student/migrations/0016_verificationattempt_status_changed.py

@@ -0,0 +1,20 @@
+# Generated by Django 4.2.15 on 2024-09-19 16:17
+
+from django.db import migrations
+import django.utils.timezone
+import model_utils.fields
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('verify_student', '0015_verificationattempt'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='verificationattempt',
+            name='status_changed',
+            field=model_utils.fields.MonitorField(default=django.utils.timezone.now, monitor='status', verbose_name='status changed'),
+        ),
+    ]

lms/djangoapps/verify_student/models.py

@@ -1214,7 +1214,7 @@ def __str__(self):
         return str(self.arguments)
 
 
-class VerificationAttempt(TimeStampedModel):
+class VerificationAttempt(TimeStampedModel, StatusModel):
     """
     The model represents impelementation-agnostic information about identity verification (IDV) attempts.
 
@@ -1251,3 +1251,8 @@ def retire_user(cls, user_id):
         """
         verification_attempts = cls.objects.filter(user_id=user_id)
         verification_attempts.delete()
+
+    @classmethod
+    def should_display_status_to_user(cls):
+        """Temporary placeholder so that calls to this method do not break edx-platform"""
+        return False