fix: make properties optional

FrigadeHQ · Nov 28, 2024 · 30910fe · 30910fe
1 parent 3ba964a
commit 30910fe
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/.changeset/blue-emus-deliver.md b/.changeset/blue-emus-deliver.md
@@ -0,0 +1,5 @@
+---
+'trench-js': patch
+---
+
+Make all `properties` optional
diff --git a/packages/trench-js/src/index.ts b/packages/trench-js/src/index.ts
@@ -82,11 +82,11 @@ class Trench {
     }
   }
 
-  track(event: string, properties: Record<string, unknown>) {
+  track(event: string, properties?: Record<string, unknown>) {
     this.analytics.track(event, properties);
   }
 
-  page(properties: Record<string, unknown>) {
+  page(properties?: Record<string, unknown>) {
     const mergedProperties = {
       referrer: typeof document !== 'undefined' ? document.referrer : undefined,
       userAgent: typeof navigator !== 'undefined' ? navigator.userAgent : undefined,
@@ -95,11 +95,11 @@ class Trench {
     this.analytics.page(mergedProperties);
   }
 
-  identify(userId: string, traits: Record<string, unknown>) {
+  identify(userId: string, traits?: Record<string, unknown>) {
     this.analytics.identify(userId, traits);
   }
 
-  group(groupId: string, traits: Record<string, unknown>) {
+  group(groupId: string, traits?: Record<string, unknown>) {
     // @ts-ignore
     this.analytics.plugins.trench.group(groupId, traits);
   }