optional mapping

elbwalker · Jul 22, 2024 · 65985fd · 65985fd
1 parent 6045277
commit 65985fd
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/packages/clients/walkerjs/src/__tests__/destination.test.ts b/packages/clients/walkerjs/src/__tests__/destination.test.ts
@@ -476,7 +476,7 @@ describe('Destination', () => {
 
     jest.clearAllMocks();
 
-    destinationIgnore.config.mapping!.foo!.bar.ignore = true;
+    destinationIgnore.config.mapping!.foo!.bar!.ignore = true;
     elb('foo bar');
     expect(mockPushA).toHaveBeenCalledTimes(0);
   });

diff --git a/packages/clients/walkerjs/src/lib/push.ts b/packages/clients/walkerjs/src/lib/push.ts
@@ -125,7 +125,7 @@ export function pushToDestination(
   }
 
   // Check for an active mapping for proper event handling
-  let mappingEvent: WebDestination.EventConfig;
+  let mappingEvent: undefined | WebDestination.EventConfig;
   const mapping = destination.config.mapping;
   let mappingKey = '';
 

diff --git a/packages/types/src/destination.ts b/packages/types/src/destination.ts
@@ -21,7 +21,9 @@ export interface Config<Custom = unknown, EventCustom = unknown> {
 }
 
 export interface Mapping<EventCustom> {
-  [entity: string]: undefined | { [action: string]: EventConfig<EventCustom> };
+  [entity: string]:
+    | undefined
+    | { [action: string]: undefined | EventConfig<EventCustom> };
 }
 
 export interface Batch<EventCustom> {