fix: logging an error when unit iframe fails to load

Right now we log nothing to the logging service when a unit iframe fails to load. The ErrorPage that’s shown isn’t using the ErrorBoundary, so we had no indication that something went wrong. This solves the problem closer to the source where the error originates.
openedx · May 16, 2023 · 5997b29 · 5997b29
1 parent d2de063
commit 5997b29
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/courseware/course/sequence/Unit.jsx b/src/courseware/course/sequence/Unit.jsx
@@ -251,6 +251,9 @@ const Unit = ({
               // could have given us a 4xx or 5xx response.
               if (!hasLoaded) {
                 setShowError(true);
+                logError('Unit iframe failed to load. Server possibly returned 4xx or 5xx response.', {
+                  iframeUrl,
+                });
               }
 
               window.onmessage = (e) => {