From c0f4d1f8672dd0c403f582a65505ff350dc577eb Mon Sep 17 00:00:00 2001
From: Shadi Naif <shadinaif@gmail.com>
Date: Mon, 25 Dec 2023 14:01:43 +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 14423aba..34c7c271 100644
--- a/CHANGELOG.rst
+++ b/CHANGELOG.rst
@@ -15,11 +15,16 @@ Please See the `releases tab <https://github.com/openedx/xblock-lti-consumer/rel
 
 Unreleased
 ~~~~~~~~~~
+
 * Remove Transifex calls and bundled translation files for the OEP-58 proposal.
 
   BREAKING CHANGE: This version breaks translations with Quince and earlier releases.
 
-9.9.0 (2024-01-24)
+9.10.0 - 2024-04-08
+------------------
+* Fix Data too long for column 'resource_id'. Increase column size to 255.
+
+9.9.0 - 2024-01-24
 ---------------------------
 * XBlockI18NService js translations support
 
diff --git a/lti_consumer/__init__.py b/lti_consumer/__init__.py
index 0ba711e2..62bcf216 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.9.0'
+__version__ = '9.10.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.