Better use of response detail from new server

Author: runestonetest

runestone/common/js/bookfuncs.js

@@ -99,13 +99,13 @@ function addReadingList() {
                 name: "link",
                 class: "btn btn-lg ' + 'buttonConfirmCompletion'",
                 href: nxt_link,
-                text: `Continue to page ${position + 2
-                    } of ${num_readings} in the reading assignment.`,
+                text: `Continue to page ${
+                    position + 2
+                } of ${num_readings} in the reading assignment.`,
             });
         } else {
             l = $("<div />", {
-                text:
-                    "This page is not part of the last reading assignment you visited.",
+                text: "This page is not part of the last reading assignment you visited.",
             });
         }
         $("#main-content").append(l);
@@ -195,7 +195,7 @@ class PageProgressBar {
             if (
                 val == 100.0 &&
                 $("#completionButton").text().toLowerCase() ===
-                "mark as completed"
+                    "mark as completed"
             ) {
                 $("#completionButton").click();
             }
@@ -211,23 +211,20 @@ async function handlePageSetup() {
         "Content-type": "application/json; charset=utf-8",
         Accept: "application/json",
     });
-    let data = { timezoneoffset: new Date().getTimezoneOffset() / 60 }
-    let request = new Request(
-        "/logger/set_tz_offset",
-        {
-            method: "POST",
-            body: JSON.stringify(data),
-            headers: headers,
-        }
-    );
+    let data = { timezoneoffset: new Date().getTimezoneOffset() / 60 };
+    let request = new Request("/logger/set_tz_offset", {
+        method: "POST",
+        body: JSON.stringify(data),
+        headers: headers,
+    });
     try {
         let response = await fetch(request);
-        if (response.status != 200) {
-            console.log(`Failed to set timezone! ${response.statusText}`)
+        if (!response.ok) {
+            console.error(`Failed to set timezone! ${response.statusText}`);
         }
         data = await response.json();
     } catch (e) {
-
+        console.error(`Error setting timezone ${e}`);
     }
 
     if (eBookConfig.isLoggedIn) {

runestone/common/js/runestonebase.js

@@ -99,9 +99,12 @@ export default class RunestoneBase {
             try {
                 let response = await fetch(request);
                 if (!response.ok) {
-                    throw new Error("Failed to save the log entry");
+                    let detail = await response.json();
+                    console.error(detail);
+                    throw new Error(`Failed to save the log entry ${detail}`);
+                } else {
+                    post_return = response.json();
                 }
-                post_return = response.json();
             } catch (e) {
                 if (this.isTimed) {
                     alert(
@@ -260,7 +263,7 @@ export default class RunestoneBase {
      */
     repopulateFromStorage(data) {
         // decide whether to use the server's answer (if there is one) or to load from storage
-        if (data !== null && this.shouldUseServer(data)) {
+        if (data !== null && data !== "no data" && this.shouldUseServer(data)) {
             this.restoreAnswers(data);
             this.setLocalStorage(data);
         } else {