@internetarchive/ia-book-actions 1.1.0 → 1.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@internetarchive/ia-book-actions",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "Bookreader actions for borrowable book",
   "author": "Internet Archive",
   "license": "AGPL-3.0-only",
@@ -23,7 +23,7 @@
     "deploy:gh": "gh-pages -t -d ghpages -m \"Build for $(git log --pretty=format:\"%h %an %ai %s\" -n1) [skip ci]\""
   },
   "dependencies": {
-    "@internetarchive/icon-info": "^1.2.6",
+    "@internetarchive/icon-info": "1.3.5",
     "@internetarchive/local-cache": "^0.2.1",
     "@internetarchive/modal-manager": "2.0.0",
     "@internetarchive/shared-resize-observer": "^0.2.0",
@@ -47,9 +47,9 @@
     "husky": "^7.0.0",
     "lint-staged": "^10.0.0",
     "prettier": "^2.0.4",
+    "rimraf": "^3.0.2",
     "sinon": "^13.0.2",
-    "vite": "^2.9.9",
-    "rimraf": "^3.0.2"
+    "vite": "^2.9.9"
   },
   "eslintConfig": {
     "extends": [

package/src/components/collapsible-action-group.js

@@ -133,9 +133,9 @@ export class CollapsibleActionGroup extends ActionsHandler {
     return html`
       <div
         class="${classMap({
-          actiongroup: true,
-          disabled: this.disabled,
-        })}"
+      actiongroup: true,
+      disabled: this.disabled,
+    })}"
       >
         ${this.getLoaderIcon}
         <section class="action-buttons primary">
@@ -194,12 +194,12 @@ export class CollapsibleActionGroup extends ActionsHandler {
         href="${action.url}"
         target=${action.target}
         @click=${() => {
-          this.clickHandler(
-            action.id,
-            action.analyticsEvent,
-            action?.borrowType
-          );
-        }}
+        this.clickHandler(
+          action.id,
+          action.analyticsEvent,
+          action?.borrowType
+        );
+      }}
       >
         ${action.id === 'purchaseBook' ? purchaseIcon : ''} ${action.text}
         <small>${action.subText}</small>
@@ -266,7 +266,6 @@ export class CollapsibleActionGroup extends ActionsHandler {
 
   get getPrimaryItems() {
     return this.primaryActions
-      .slice(1)
       .map(
         action =>
           html`<li>${this.renderActionButton(action, this.initialButton)}</li>`
@@ -280,9 +279,9 @@ export class CollapsibleActionGroup extends ActionsHandler {
   get getLoaderIcon() {
     return html`<img
       class="${classMap({
-        actionloader: true,
-        disabled: this.disabled,
-      })}"
+      actionloader: true,
+      disabled: this.disabled,
+    })}"
       alt=""
       src="${this.loaderIcon}"
     />`;

package/src/core/services/actions-config.js

@@ -36,7 +36,7 @@ export default class ActionsConfig {
   firstBrowseConfig() {
     return {
       id: 'browseBook',
-      text: 'Borrow',
+      text: 'Borrow (auto-renewing)',
       className: 'primary',
       analyticsEvent: {
         category: this.analyticsCategories.preview,

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

@@ -25,7 +25,7 @@ describe('Get Lending Actions', () => {
 
     expect(actions.primaryTitle).to.equal(bookTitles.available_1hr);
     expect(actions.primaryActions.length).to.equal(3);
-    expect(actions.primaryActions[0].text).to.equal('Borrow');
+    expect(actions.primaryActions[0].text).to.equal('Borrow (auto-renewing)');
     expect(actions.primaryActions[1].text).to.equal('Borrow for 14 days');
     expect(actions.secondaryActions[1].text).to.equal('Purchase at ');
   });
@@ -111,7 +111,7 @@ describe('Get Lending Actions', () => {
         expect(actions.primaryTitle).to.equal(bookTitles.claim_waitlist);
         expect(actions.primaryActions.length).to.equal(3);
         expect(actions.primaryActions[0].text).to.equal('Borrow for 14 days');
-        expect(actions.primaryActions[1].text).to.equal('Borrow');
+        expect(actions.primaryActions[1].text).to.equal('Borrow (auto-renewing)');
         expect(actions.primaryActions[2].text).to.equal('Leave Waitlist');
         expect(spy.called).to.be.true;
       });
@@ -138,7 +138,7 @@ describe('Get Lending Actions', () => {
         expect(actions.primaryTitle).to.equal(bookTitles.on_waitlist);
         expect(actions.primaryActions.length).to.equal(2);
         expect(actions.primaryActions[0].text).to.equal('Leave Waitlist');
-        expect(actions.primaryActions[1].text).to.equal('Borrow');
+        expect(actions.primaryActions[1].text).to.equal('Borrow (auto-renewing)');
         expect(claimSpy.called).to.be.false;
         expect(waitlistSpy.called).to.be.true;
       });
@@ -162,7 +162,7 @@ describe('Get Lending Actions', () => {
         expect(actions.primaryTitle).to.equal(bookTitles.on_waitlist);
         expect(actions.primaryActions.length).to.equal(2);
         expect(actions.primaryActions[0].text).to.equal('Leave Waitlist');
-        expect(actions.primaryActions[1].text).to.equal('Borrow');
+        expect(actions.primaryActions[1].text).to.equal('Borrow (auto-renewing)');
         expect(waitlistSpy.called).to.be.true;
       });
     });

package/test/ia-book-actions.test.js

@@ -149,12 +149,12 @@ describe('Primary Actions data', () => {
     const expectedPrimaryActions = [
       {
         text: 'Borrow', // changed from 'Borrow for 1 hour'
-        callback: () => {},
+        callback: () => { },
         className: 'ia-button primary',
       },
       {
         text: 'Borrow for 14 days',
-        callback: () => {},
+        callback: () => { },
         className: 'ia-button primary',
         disabled: false,
       },
@@ -228,7 +228,7 @@ describe('Borrow status actions', () => {
 
     // removed available_to_borrow from lending bar
     expect(el.lendingStatus.available_to_borrow).to.be.false;
-    expect(el.primaryActions[0].text).to.equal('Borrow');
+    expect(el.primaryActions[0].text).to.equal('Borrow (auto-renewing)');
   });
 
   it('Check action for borrowable book without user', async () => {
@@ -244,7 +244,7 @@ describe('Borrow status actions', () => {
     const expectedPrimaryActions = [
       {
         text: 'Log In and Borrow',
-        callback: () => {},
+        callback: () => { },
         className: 'ia-button danger',
       },
     ];