diff --git a/CHANGELOG.rst b/CHANGELOG.rst index b66392b6bb3..faa1603d1f3 100644 --- a/CHANGELOG.rst +++ b/CHANGELOG.rst @@ -14,6 +14,10 @@ Change Log Unreleased ~~~~~~~~~~ +[4.3.1] - 2021-10-28 +~~~~~~~~~~~~~~~~~~~~ +* Set the to be retired column video_url on ProctoredExamSoftwareSecureReview to be nullable. + [4.3.0] - 2021-10-28 ~~~~~~~~~~~~~~~~~~~~ * Upgrade the requirements and move edx-proctoring to be on Django 3.2 instead of Django 2.2 diff --git a/edx_proctoring/__init__.py b/edx_proctoring/__init__.py index 1c1ca31b2f1..6544bdb4edd 100644 --- a/edx_proctoring/__init__.py +++ b/edx_proctoring/__init__.py @@ -3,6 +3,6 @@ """ # Be sure to update the version number in edx_proctoring/package.json -__version__ = '4.3.0' +__version__ = '4.3.1' default_app_config = 'edx_proctoring.apps.EdxProctoringConfig' # pylint: disable=invalid-name diff --git a/edx_proctoring/admin.py b/edx_proctoring/admin.py index cfe2499bae7..8ff858212ee 100644 --- a/edx_proctoring/admin.py +++ b/edx_proctoring/admin.py @@ -313,13 +313,6 @@ def save_model(self, request, obj, form, change): obj.reviewed_by = request.user obj.save() - def get_form(self, request, obj=None, change=False, **kwargs): - """ Returns software secure review form """ - form = super().get_form(request, obj, change, **kwargs) - if 'video_url' in form.base_fields: - del form.base_fields['video_url'] - return form - def lookup_allowed(self, lookup, value): """ Checks if lookup allowed or not """ if lookup == 'exam__course_id': diff --git a/edx_proctoring/migrations/0020_auto_20211028_1915.py b/edx_proctoring/migrations/0020_auto_20211028_1915.py new file mode 100644 index 00000000000..602abf2519d --- /dev/null +++ b/edx_proctoring/migrations/0020_auto_20211028_1915.py @@ -0,0 +1,23 @@ +# Generated by Django 2.2.24 on 2021-10-28 19:15 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('edx_proctoring', '0019_proctoredexamsoftwaresecurereview_encrypted_video_url'), + ] + + operations = [ + migrations.AlterField( + model_name='proctoredexamsoftwaresecurereview', + name='video_url', + field=models.TextField(null=True), + ), + migrations.AlterField( + model_name='proctoredexamsoftwaresecurereviewhistory', + name='video_url', + field=models.TextField(null=True), + ), + ] diff --git a/edx_proctoring/models.py b/edx_proctoring/models.py index e54eaa5ad36..35ae9508d1b 100644 --- a/edx_proctoring/models.py +++ b/edx_proctoring/models.py @@ -774,7 +774,7 @@ class ProctoredExamSoftwareSecureReview(TimeStampedModel): # URL for the exam video that had been reviewed # NOTE: To be deleted in future release, once the code that depends on it # has been removed - video_url = models.TextField() + video_url = models.TextField(null=True) # Encrypted URL for the exam video that had been reviewed encrypted_video_url = models.BinaryField(null=True) @@ -850,7 +850,7 @@ class ProctoredExamSoftwareSecureReviewHistory(TimeStampedModel): raw_data = models.TextField() # URL for the exam video that had been reviewed - video_url = models.TextField() + video_url = models.TextField(null=True) # user_id of person who did the review (can be None if submitted via server-to-server API) reviewed_by = models.ForeignKey(USER_MODEL, null=True, related_name='+', on_delete=models.CASCADE) diff --git a/package.json b/package.json index cfaff6b8541..2489c97d9c0 100644 --- a/package.json +++ b/package.json @@ -1,7 +1,7 @@ { "name": "@edx/edx-proctoring", "//": "Note that the version format is slightly different than that of the Python version when using prereleases.", - "version": "4.3.0", + "version": "4.3.1", "main": "edx_proctoring/static/index.js", "scripts": { "test": "gulp test"