🔧(sandbox) allowing multiple forums for a same LTIContext on sandbox

In development mode, the sandbox allows using Ashley via a dummy LTI consumer. So far, we can’t create several forums for the same course, since the forum LTI id is generated from the course ID. This prevents us from testing features that we are trying to implement. Enabling to set the LTI id value will fix this limit.
openfun · Jun 21, 2021 · f5338ec · f5338ec
1 parent 83ccbf6
commit f5338ec
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -25,6 +25,8 @@ Versioning](https://semver.org/spec/v2.0.0.html).
  - add frontend internationalisation and react components autoload
  - add django rest framework to promote/revoke moderators for current LTIContext
  - add new role moderator and permission to manage moderators
+ - add a `LTI ID` field in the sandbox settings to be able to have multiple
+   forums for a same LTIContext
 
 ### Fixed
  - fix sorting on sticky and announcements topics

diff --git a/sandbox/dev_tools/forms.py b/sandbox/dev_tools/forms.py
@@ -1,5 +1,7 @@
 """Forms definition for the dev_tools application."""
 
+import uuid
+
 from django import forms
 from django.utils.translation import gettext_lazy as _
 from lti_toolbox.models import LTIPassport
@@ -34,6 +36,11 @@ class LTIConsumerForm(forms.Form):
         label="Course Title", max_length=100, initial="Mathematics 101"
     )
 
+    forum_lti_id = forms.UUIDField(
+        label="LTI ID",
+        initial=uuid.uuid5(uuid.NAMESPACE_DNS, "default"),
+    )
+
     role = forms.ChoiceField(
         choices=(
             ("Student", _("Student")),

diff --git a/sandbox/dev_tools/views.py b/sandbox/dev_tools/views.py
@@ -28,11 +28,7 @@ def dev_consumer(request: HttpRequest) -> HttpResponse:
             launch_url = request.build_absolute_uri(
                 reverse(
                     "forum.lti.view",
-                    kwargs={
-                        "uuid": uuid.uuid5(
-                            uuid.NAMESPACE_DNS, form.cleaned_data["course_id"]
-                        )
-                    },
+                    kwargs={"uuid": form.cleaned_data["forum_lti_id"]},
                 )
             )
             lti_params = _generate_signed_parameters(form, launch_url, passport)