@internetarchive/ia-book-actions 0.0.27-a1 → 0.0.28

package/demo/index.html

@@ -174,8 +174,10 @@
     document.querySelectorAll('.printDisabled input[type=checkbox]').forEach((item) => {
       item.addEventListener('click', (e) => {
         if (e.target.checked) {
+          if (e.target.value === 'is_printdisabled') lendingStatus1.is_printdisabled = true;
           if (e.target.value === 'user_is_printdisabled') lendingStatus1.user_is_printdisabled = true;
         } else {
+          if (e.target.value === 'is_printdisabled') lendingStatus1.is_printdisabled = false;
           if (e.target.value === 'user_is_printdisabled') lendingStatus1.user_is_printdisabled = false;
         }
 
@@ -277,6 +279,7 @@
               <label><input type="checkbox" name="user" value="isLoggedIn" />isLoggedIn</label>
             </td>
             <td class="printDisabled">
+              <label><input type="checkbox" name="is_printdisabled" value="is_printdisabled" />is_printdisabled</label>
               <label><input type="checkbox" name="user_is_printdisabled" value="user_is_printdisabled" />user_is_printdisabled</label>
             </td>
             <td class="availableToBrowse">

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@internetarchive/ia-book-actions",
-  "version": "0.0.27-a1",
+  "version": "0.0.28",
   "description": "Bookreader actions for borrowable book",
   "author": "Internet Archive",
   "license": "AGPL-3.0-only",

package/src/core/services/get-lending-actions.js

@@ -146,17 +146,9 @@ export class GetLendingActions {
   }
 
   borrowPrintDisabledAction() {
-    const lendingStatus = this.lendingStatus || {};
-    const browsingHasExpired =
-      !lendingStatus.available_to_browse && lendingStatus.browsingExpired;
-
-    // one hour borrow config
-    const oneHrBorrow = browsingHasExpired
-      ? this.actionsConfig.browseAgainConfig()
-      : this.actionsConfig.firstBrowseConfig();
     return {
       primaryTitle: bookTitles.eligible_pd,
-      primaryActions: [oneHrBorrow, this.actionsConfig.borrowBookConfig()],
+      primaryActions: [this.actionsConfig.borrowBookConfig()],
       primaryColor: 'primary',
       secondaryActions: [
         this.actionsConfig.adminAccessConfig(),
@@ -409,7 +401,8 @@ export class GetLendingActions {
     const notBorrowable =
       !lendingStatus.available_to_borrow && !lendingStatus.available_to_browse;
 
-    const userCanAccessPrintDisabled = lendingStatus.user_is_printdisabled;
+    const userCanAccessPrintDisabled =
+      lendingStatus.is_printdisabled && lendingStatus.user_is_printdisabled;
 
     const canBorrow =
       (lendingStatus.available_to_browse ||

package/src/core/services/loan-token-poller.js

@@ -21,9 +21,14 @@ export class LoanTokenPoller {
   disconnectedInterval() {
     clearInterval(this.loanTokenInterval);
     this.loanTokenInterval = undefined;
-    if (window.Sentry) {
-      window?.Sentry?.captureMessage('loan token interval cleared');
-    }
+
+    // temporary disabling sentry notice
+    // as message is informational, not critical nor an error.
+    // also, as of 11/28/2022, JS build maps are not working properly on petabox
+    // and as a result, this comment is not properly grouped.
+    // if (window.Sentry) {
+    //   window?.Sentry?.captureMessage('loan token interval cleared');
+    // }
   }
 
   async enableBookAccess() {
@@ -83,7 +88,10 @@ export class LoanTokenPoller {
       identifier: this.identifier,
       action,
       error: data => {
-        window?.Sentry?.captureMessage('handleLoanTokenPoller error');
+        window?.Sentry?.captureMessage(
+          'handleLoanTokenPoller error',
+          JSON.stringify(data)
+        );
         this.disconnectedInterval(); // stop token fetch api
         this.errorCallback({ detail: { action, data } });
         // send LendingServiceError to GA