SpeedGrader in VS launches

hypothesis · Nov 15, 2021 · 0d95d5b · 0d95d5b
1 parent a0b7980
commit 0d95d5b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lms/static/scripts/frontend_apps/components/BasicLTILaunchApp.js b/lms/static/scripts/frontend_apps/components/BasicLTILaunchApp.js
@@ -216,8 +216,8 @@ export default function BasicLTILaunchApp() {
       return;
     }
 
-    // Don't report a submission until the URL has been successfully fetched.
-    if (!contentUrl) {
+    // Don't report a submission until we have the data needed to display the assignment content
+    if (!contentUrl && !vitalSourceConfig) {
       return;
     }
     try {
@@ -233,7 +233,7 @@ export default function BasicLTILaunchApp() {
       // submission.
       handleError(e, 'error-reporting-submission', false);
     }
-  }, [authToken, canvas.speedGrader, contentUrl]);
+  }, [authToken, canvas.speedGrader, contentUrl, vitalSourceConfig]);
 
   useEffect(() => {
     reportSubmission();

diff --git a/lms/validation/_lti_launch_params.py b/lms/validation/_lti_launch_params.py
@@ -118,6 +118,7 @@ def _decode_url(self, _data, **_kwargs):  # pylint:disable=no-self-use
             url.lower().startswith("http%3a")
             or url.lower().startswith("https%3a")
             or url.lower().startswith("canvas%3a")
+            or url.lower().startswith("vitalsource%3a")
         ):
             url = unquote(url)
             _data["url"] = url