fix: [CO-7] fix lti launch url on external (#18)

raccoongang · Feb 26, 2025 · c53ed99 · c53ed99
1 parent c2580a1
commit c53ed99
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -16,6 +16,10 @@ Please See the `releases tab <https://github.com/openedx/xblock-lti-consumer/rel
 Unreleased
 ~~~~~~~~~~
 
+9.13.1 - 2025-02-26
+-------------------
+* Fixed taken lti launch url for external LTI configurations.
+
 9.13.0 - 2025-02-25
 -------------------
 * Added ability to override launch URL on block for external LTI configurations.

diff --git 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.13.0'
+__version__ = '9.13.1'
diff --git a/lti_consumer/models.py b/lti_consumer/models.py
@@ -589,7 +589,7 @@ def _get_lti_1p3_consumer(self):
             consumer = consumer_class(
                 iss=get_lti_api_base(),
                 lti_oidc_url=self.external_config.get('lti_1p3_oidc_url'),
-                lti_launch_url=self.lti_1p3_launch_url or self.external_config.get('lti_1p3_launch_url'),
+                lti_launch_url=block.lti_1p3_launch_url or self.external_config.get('lti_1p3_launch_url'),
                 client_id=self.external_config.get('lti_1p3_client_id'),
                 # Deployment ID hardcoded to 1 since
                 # we're not using multi-tenancy.