Merge PR #136 open-craft/kshitij/fix-migrations

* Commits:
  Fix migration `config_id` conflict.

README.rst

@@ -298,6 +298,11 @@ Please do not report security issues in public. Send security concerns via email
 Changelog
 =========
 
+2.5.2 - 2021-01-20
+------------------
+
+* Fix issue with migration that causes migration failure due to duplicate `config_id` values.
+
 2.5.1 - 2021-01-19
 ------------------
 

lti_consumer/migrations/0006_add_on_model_config_for_lti_1p1.py

@@ -6,6 +6,13 @@ import jsonfield.fields
 import uuid
 
 
+def create_config_ids(apps, schema_editor):
+    LtiConfiguration = apps.get_model('lti_consumer', 'LtiConfiguration')
+    for config in LtiConfiguration.objects.all():
+        config.config_id = uuid.uuid4()
+        config.save()
+
+
 class Migration(migrations.Migration):
 
     dependencies = [
@@ -14,6 +21,12 @@ class Migration(migrations.Migration):
 
     operations = [
         migrations.AddField(
+            model_name='lticonfiguration',
+            name='config_id',
+            field=models.UUIDField(default=uuid.uuid4, null=True),
+        ),
+        migrations.RunPython(create_config_ids),
+        migrations.AlterField(
             model_name='lticonfiguration',
             name='config_id',
             field=models.UUIDField(default=uuid.uuid4, editable=False, unique=True),

setup.py

@@ -49,7 +49,7 @@ with open('README.rst') as _f:
 
 setup(
     name='lti-consumer-xblock',
-    version='2.5.1',
+    version='2.5.2',
     description='This XBlock implements the consumer side of the LTI specification.',
     long_description=long_description,
     long_description_content_type='text/x-rst',