From 6245ed2fd204f1810e36f990503f6ad1a5f3b35f Mon Sep 17 00:00:00 2001 From: Muhammad Faraz Maqsood Date: Fri, 13 Dec 2024 13:02:43 +0500 Subject: [PATCH] docs: update lti_url for testing LTI 1.1/1.2 - update lti testing url in the README, as saltire has updated the test URLs. - add a note in the README to use `norefresh` param, to avoid failures if using more than one same LTI xblocks in the same unit. Details can be viewed here: https://github.com/openedx/xblock-lti-consumer/issues/379 - close #379. --- README.rst | 6 +++++- lti_consumer/lti_xblock.py | 4 ++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/README.rst b/README.rst index e43c1ce2..83d99696 100644 --- a/README.rst +++ b/README.rst @@ -160,7 +160,11 @@ http://edx.readthedocs.io/projects/open-edx-building-and-running-a-course/en/lat as seen above). Make a unit, select "Advanced", then "LTI Consumer". 3. Click edit and fill in the following fields: ``LTI ID``: "test" - ``LTI URL``: "https://lti.tools/saltire/tp" + ``LTI URL``: "https://saltire.lti.app/tool" + **Note:** If you are using more than one same LTI xblocks in the same unit, + please append the `norefresh` parameter to the LTI URL to avoid any + potential failures. Then LTI URL will look like this: + `https://saltire.lti.app/tool?norefresh`. 4. Click save. The unit should refresh and you should see "Passed" in the "Verification" field of the message tab in the LTI Tool Provider emulator. 5. Click the "Publish" button. diff --git a/lti_consumer/lti_xblock.py b/lti_consumer/lti_xblock.py index 08789041..3978ae8f 100644 --- a/lti_consumer/lti_xblock.py +++ b/lti_consumer/lti_xblock.py @@ -646,7 +646,7 @@ def workbench_scenarios(): ask_to_send_email="False" enable_processors="True" launch_target="new_window" - launch_url="https://lti.tools/saltire/tp" /> + launch_url="https://saltire.lti.app/tool?norefresh" /> + launch_url="https://saltire.lti.app/tool?norefresh" /> '''), ]