Include display_name in the Canvas custom variables

hypothesis · Feb 6, 2024 · 3601e82 · 3601e82
1 parent 79cbeaf
commit 3601e82
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/lms/templates/config.xml.jinja2 b/lms/templates/config.xml.jinja2
@@ -18,6 +18,7 @@ http://www.imsglobal.org/xsd/lti/ltiv1p0/imslticp_v1p0.xsd">
   <blti:custom>
     <lticm:property name="custom_canvas_course_id">$Canvas.course.id</lticm:property>
     <lticm:property name="custom_canvas_api_domain">$Canvas.api.domain</lticm:property>
+    <lticm:property name="custom_display_name">$Person.name.display</lticm:property>
   </blti:custom>
   <blti:extensions platform="canvas.instructure.com">
     <lticm:options name="link_selection">

diff --git a/lms/views/canvas/config.py b/lms/views/canvas/config.py
@@ -80,6 +80,7 @@ def config_json(request):
             "custom_canvas_course_id": "$Canvas.course.id",
             "custom_canvas_api_domain": "$Canvas.api.domain",
             "custom_canvas_user_id": "$Canvas.user.id",
+            "custom_display_name": "$Person.name.display",
         },
         "scopes": [
             "https://purl.imsglobal.org/spec/lti-ags/scope/score",

diff --git a/tests/unit/lms/views/canvas/config_test.py b/tests/unit/lms/views/canvas/config_test.py
@@ -49,6 +49,7 @@ def test_it(self, pyramid_request):
                 "custom_canvas_course_id": "$Canvas.course.id",
                 "custom_canvas_api_domain": "$Canvas.api.domain",
                 "custom_canvas_user_id": "$Canvas.user.id",
+                "custom_display_name": "$Person.name.display",
             },
             "scopes": [
                 "https://purl.imsglobal.org/spec/lti-ags/scope/score",