Skip to content

Commit

Permalink
Merge pull request #523 from edly-io/fix/short_resource_id
Browse files Browse the repository at this point in the history
fix: Data too long for column 'resource_id'
  • Loading branch information
feanil authored Jan 22, 2025
2 parents beab598 + d245292 commit 68e3d6e
Show file tree
Hide file tree
Showing 4 changed files with 29 additions and 3 deletions.
10 changes: 9 additions & 1 deletion CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,10 @@ Please See the `releases tab <https://github.com/openedx/xblock-lti-consumer/rel
Unreleased
~~~~~~~~~~

9.13.2 - 2025-01-21
-------------------
* Fix Data too long for column 'resource_id'. Increase column size to 255.

9.13.1 - 2025-01-15
-------------------
* Fix broken LTI 1.3 launch by adding a `kid` field to both the JWT header and data returned by the public keyset endpoint.
Expand All @@ -24,6 +28,10 @@ Unreleased
-------------------
* Removed pyjwkset package and replace with pyjwt package

9.12.1 - 2024-12-19
-------------------
* Update lti_url in readme for testing LTI 1.1/1.2.

9.12.0 - 2024-11-14
-------------------
* Dropped support for Python 3.8 and added support for Python 3.12.
Expand Down Expand Up @@ -53,7 +61,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

Expand Down
2 changes: 1 addition & 1 deletion lti_consumer/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,4 @@
from .apps import LTIConsumerApp
from .lti_xblock import LtiConsumerXBlock

__version__ = '9.13.1'
__version__ = '9.13.2'
18 changes: 18 additions & 0 deletions lti_consumer/migrations/0018_increase_length_of_resource_id.py
Original file line number Diff line number Diff line change
@@ -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),
),
]
2 changes: 1 addition & 1 deletion lti_consumer/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down

0 comments on commit 68e3d6e

Please sign in to comment.