Merge pull request #742 from B77Mills/monorailBrowserIndex

Correctly apply browser/index.js config overrides and fallback logic
parameter1 · Jul 20, 2023 · a13e526 · a13e526
2 parents 1b343f9 + 220311e
commit a13e526
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/packages/marko-web-theme-monorail/browser/index.js b/packages/marko-web-theme-monorail/browser/index.js
@@ -28,19 +28,25 @@ const setP1EventsIdentity = ({ p1events, brandKey, encryptedId }) => {
   p1events('setIdentity', `omeda.${brandKey}.customer*${encryptedId}~encrypted`);
 };
 
-export default (Browser, config = {
+const defaultConfig = {
   enableOmedaIdentityX: true,
-  withGTM: true,
   withP1Events: true,
   idxArgs: {},
   inquiryArgs: {},
-}) => {
+};
+
+export default (Browser, configOverrides = {}) => {
+  const config = { ...defaultConfig, ...configOverrides };
   const { EventBus } = Browser;
-  const { enableOmedaIdentityX } = config;
+  const {
+    enableOmedaIdentityX,
+    withP1Events,
+  } = config;
+
   const idxArgs = config.idxArgs || {};
   const inquiryArgs = config.inquiryArgs || {};
 
-  if (config.withP1Events) {
+  if (withP1Events) {
     P1Events(Browser);
   }