From df5fd9ba3a054659589fd1711a06323a508cb5d8 Mon Sep 17 00:00:00 2001
From: Shadi Naif <shadinaif@gmail.com>
Date: Mon, 8 Apr 2024 15:26:58 +0300
Subject: [PATCH] fix: Data too long for column 'resource_id'

when the course id is a little bit long, the lti xblock id becomes too long for resource_id to handle
---
 CHANGELOG.rst                                  |  7 ++++++-
 lti_consumer/__init__.py                       |  2 +-
 .../0018_increase_length_of_resource_id.py     | 18 ++++++++++++++++++
 lti_consumer/models.py                         |  2 +-
 4 files changed, 26 insertions(+), 3 deletions(-)
 create mode 100644 lti_consumer/migrations/0018_increase_length_of_resource_id.py

diff --git a/CHANGELOG.rst b/CHANGELOG.rst
index 77512030..0a47cc5d 100644
--- a/CHANGELOG.rst
+++ b/CHANGELOG.rst
@@ -16,6 +16,11 @@ Please See the `releases tab <https://github.com/openedx/xblock-lti-consumer/rel
 Unreleased
 ~~~~~~~~~~
 
+
+9.12.0 - 2024-07-16
+-------------------
+* Fix Data too long for column 'resource_id'. Increase column size to 255.
+
 9.11.3 - 2024-05-30
 -------------------
 * Remove utf-8 encoding for user full name
@@ -38,7 +43,7 @@ Unreleased
 * BREAKING CHANGE: This version breaks translations with Quince and earlier releases.
 * Adding python 3.11 support. Dropping django32 support.
 
-9.9.0 (2024-01-24)
+9.9.0 - 2024-01-24
 ---------------------------
 * XBlockI18NService js translations support
 
diff --git a/lti_consumer/__init__.py b/lti_consumer/__init__.py
index e85c6da7..4861471f 100644
--- a/lti_consumer/__init__.py
+++ b/lti_consumer/__init__.py
@@ -4,4 +4,4 @@
 from .apps import LTIConsumerApp
 from .lti_xblock import LtiConsumerXBlock
 
-__version__ = '9.11.3'
+__version__ = '9.12.0'
diff --git a/lti_consumer/migrations/0018_increase_length_of_resource_id.py b/lti_consumer/migrations/0018_increase_length_of_resource_id.py
new file mode 100644
index 00000000..c55ea08c
--- /dev/null
+++ b/lti_consumer/migrations/0018_increase_length_of_resource_id.py
@@ -0,0 +1,18 @@
+# Generated by Django 3.2.23 on 2023-12-25 10:56
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('lti_consumer', '0017_lticonfiguration_lti_1p3_redirect_uris'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='ltiagslineitem',
+            name='resource_id',
+            field=models.CharField(blank=True, max_length=255),
+        ),
+    ]
diff --git a/lti_consumer/models.py b/lti_consumer/models.py
index 6cb35f3c..5bfd38eb 100644
--- a/lti_consumer/models.py
+++ b/lti_consumer/models.py
@@ -695,7 +695,7 @@ class LtiAgsLineItem(models.Model):
     )
 
     # Tool resource identifier, not used by the LMS.
-    resource_id = models.CharField(max_length=100, blank=True)
+    resource_id = models.CharField(max_length=255, blank=True)
 
     # LMS Resource link
     # Must be the same as the one sent in the tool's LTI launch.