@ordergroove/offers 2.36.2-alpha-PR-836-2.3 → 2.37.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ordergroove/offers",
-  "version": "2.36.2-alpha-PR-836-2.3+2e663ca9",
+  "version": "2.37.0",
   "description": "offer state component",
   "author": "Eugenio Lattanzio <eugenio63@gmail.com>",
   "homepage": "https://github.com/ordergroove/plush-toys#readme",
@@ -48,5 +48,5 @@
     "@ordergroove/offers-templates": "^0.9.6",
     "@types/lodash.memoize": "^4.1.9"
   },
-  "gitHead": "2e663ca9c0463a9d7d8040066c32aa187f72c25d"
+  "gitHead": "e702e182ecc53ced69c632d6269bb8f98c778b42"
 }

package/src/components/Offer.js

@@ -28,16 +28,24 @@ import { DEFAULT_OFFER_MODULE } from '../core/constants';
 
 const memoizeKey = (...args) => JSON.stringify(args);
 
-let hasLogged = false;
-function logMulticurrencyWarning(storeCurrency, primaryCurrency) {
-  // only log once
-  if (!hasLogged) {
-    console.warn(
-      `Hiding Ordergroove offer since the store currency ${storeCurrency} does not match your configured currency ${primaryCurrency} and you are not set up for multicurrency. Contact your Ordergroove representative for next steps.`
-    );
-    hasLogged = true;
-  }
-}
+const logOnce = messageFn => {
+  let hasLogged = false;
+  return (...args) => {
+    if (!hasLogged) {
+      console.warn(messageFn(...args));
+      hasLogged = true;
+    }
+  };
+};
+
+const logMulticurrencyWarning = logOnce(
+  (storeCurrency, primaryCurrency) =>
+    `Hiding Ordergroove offer since the store currency ${storeCurrency} does not match your configured currency ${primaryCurrency} and you are not set up for multicurrency. Contact your Ordergroove representative for next steps.`
+);
+
+const logProductSpecificFrequencyListWarning = logOnce(
+  () => `Hiding Ordergroove offer since cart offers does not currently support product-specific frequency lists.`
+);
 
 export const productAndComponents = memoize(
   (product, components) => Object.assign({ components }, product),
@@ -327,8 +335,14 @@ export class Offer extends TemplateElement {
         this.config.storeCurrency === this.config.merchantSettings.currency_code;
       if (!shouldEnable) {
         logMulticurrencyWarning(this.config.storeCurrency, this.config.merchantSettings.currency_code);
+        return false;
       }
-      return shouldEnable;
+    }
+
+    // product-specific frequency lists are not supported in cart offers
+    if (this.isCart && this.config?.hasProductSpecificFrequencies) {
+      logProductSpecificFrequencyListWarning();
+      return false;
     }
 
     return true;

package/src/components/__tests__/Offer.spec.js

@@ -302,25 +302,25 @@ describe('previewMode', () => {
   });
 });
 
-describe('multi-currency', () => {
-  async function getOfferElement(config) {
-    const element = new Offer();
-    element.innerHTML = `
-      <p>Offer content</p>
-    `;
-    element.config = config;
-    await appendToBody(element);
-    return element;
-  }
-
-  function assertOfferShown(element) {
-    expect(element.querySelector('*').assignedSlot).not.toBeNull();
-  }
-
-  function assertOfferHidden(element) {
-    expect(element.querySelector('*').assignedSlot).toBeNull();
-  }
+async function getOfferElement(config) {
+  const element = new Offer();
+  element.innerHTML = `
+    <p>Offer content</p>
+  `;
+  element.config = config;
+  await appendToBody(element);
+  return element;
+}
+
+function assertOfferShown(element) {
+  expect(element.querySelector('*').assignedSlot).not.toBeNull();
+}
+
+function assertOfferHidden(element) {
+  expect(element.querySelector('*').assignedSlot).toBeNull();
+}
 
+describe('multi-currency', () => {
   it('should show the offer when config not set', async () => {
     const element = await getOfferElement();
     assertOfferShown(element);
@@ -369,3 +369,22 @@ describe('multi-currency', () => {
     assertOfferShown(element);
   });
 });
+
+describe('product specific frequencies', () => {
+  it('should hide the offer when on cart', async () => {
+    const element = await getOfferElement({
+      hasProductSpecificFrequencies: true
+    });
+    element.isCart = true;
+    await element.updateComplete;
+
+    assertOfferHidden(element);
+  });
+
+  it('should show the offer when not on cart', async () => {
+    const element = await getOfferElement({
+      hasProductSpecificFrequencies: true
+    });
+    assertOfferShown(element);
+  });
+});

package/src/shopify/__tests__/shopifyReducer.spec.js

@@ -337,34 +337,37 @@ describe('config', () => {
     );
   });
 
+  const defaultSellingPlanPayload = {
+    product: {
+      selling_plan_groups: [
+        {
+          name: DEFAULT_PAY_AS_YOU_GO_GROUP_NAME,
+          selling_plans: [
+            {
+              id: 'yum selling plan id 1'
+            },
+            {
+              id: 'yum selling plan id 2'
+            }
+          ]
+        }
+      ]
+    }
+  };
+
   it('should return selling plan ids as frequencies given action SETUP_PRODUCT', () => {
     const actual = config(
       {},
       {
         type: constants.SETUP_PRODUCT,
-        payload: {
-          product: {
-            selling_plan_groups: [
-              {
-                name: DEFAULT_PAY_AS_YOU_GO_GROUP_NAME,
-                selling_plans: [
-                  {
-                    id: 'yum selling plan id 1'
-                  },
-                  {
-                    id: 'yum selling plan id 2'
-                  }
-                ]
-              }
-            ]
-          }
-        }
+        payload: defaultSellingPlanPayload
       }
     );
 
     expect(actual).toEqual(
       jasmine.objectContaining({
-        frequencies: ['yum selling plan id 1', 'yum selling plan id 2']
+        frequencies: ['yum selling plan id 1', 'yum selling plan id 2'],
+        hasProductSpecificFrequencies: false
       })
     );
   });
@@ -462,7 +465,26 @@ describe('config', () => {
 
     expect(actual).toEqual(
       jasmine.objectContaining({
-        frequencies: ['psfl-id-1', 'psfl-id-2', 'psfl-id-3']
+        frequencies: ['psfl-id-1', 'psfl-id-2', 'psfl-id-3'],
+        hasProductSpecificFrequencies: true
+      })
+    );
+  });
+
+  it('does not overwrite hasProductSpecificFrequencies on subsequent calls', () => {
+    const actual = config(
+      {
+        hasProductSpecificFrequencies: true
+      },
+      {
+        type: constants.SETUP_PRODUCT,
+        payload: defaultSellingPlanPayload
+      }
+    );
+
+    expect(actual).toEqual(
+      jasmine.objectContaining({
+        hasProductSpecificFrequencies: true
       })
     );
   });

package/src/shopify/shopifyReducer.js

@@ -127,8 +127,8 @@ export const reduceNewOptinsFromOfferResponse = (
   }, []);
 
 const getOGSellingPlanGroup = product => {
-  const productSpecificFrequencySellingPlanGroup = product?.selling_plan_groups.find(group =>
-    group.name.startsWith('og_psfl')
+  const productSpecificFrequencySellingPlanGroup = product?.selling_plan_groups.find(
+    isProductSpecificFrequencySellingPlanGroup
   );
 
   return (
@@ -137,6 +137,8 @@ const getOGSellingPlanGroup = product => {
   );
 };
 
+const isProductSpecificFrequencySellingPlanGroup = group => group.name.startsWith('og_psfl');
+
 const productOrVariantInStockReducer = (acc, cur) => ({
   ...overrideLineKey(acc, cur.id, cur.available),
   [cur.id]: cur.available
@@ -285,7 +287,9 @@ export const config = (
         frequencies,
         frequenciesEveryPeriod,
         frequenciesText,
-        ...(defaultFrequency ? { defaultFrequency } : {})
+        ...(defaultFrequency ? { defaultFrequency } : {}),
+        hasProductSpecificFrequencies:
+          state.hasProductSpecificFrequencies || isProductSpecificFrequencySellingPlanGroup(sellingPlanGroup)
       };
     }