add check for json stringifiable properties

Author: fadi-george

examples/RNOneSignalTS/OSButtons.tsx

@@ -435,7 +435,7 @@ class OSButtons extends React.Component<Props> {
       loggingFunction('Tracking event: ', 'ReactNative');
       const platform = Platform.OS; // This will be 'ios' or 'android'
       OneSignal.User.trackEvent(`ReactNative-${platform}`, {
-        ABC: '123',
+        DEF: '456',
       });
     });
 

src/index.ts

@@ -609,10 +609,14 @@ export namespace OneSignal {
     /** Track custom events for the current user. */
     export function trackEvent(
       name: string,
-      properties?: Record<string, string>,
+      properties?: Record<string, unknown>,
     ) {
       if (!isNativeModuleLoaded(RNOneSignal)) return;
 
+      if (!isObjectSerializable(properties)) {
+        return console.error('Properties must be JSON-serializable');
+      }
+
       RNOneSignal.trackEvent(name, properties);
     }
   }
@@ -1010,6 +1014,21 @@ export namespace OneSignal {
   }
 }
 
+/**
+ * Returns true if the value is a JSON-serializable object.
+ */
+function isObjectSerializable(value: unknown): boolean {
+  if (!(typeof value === 'object' && value !== null && !Array.isArray(value))) {
+    return false;
+  }
+  try {
+    JSON.stringify(value);
+    return true;
+  } catch (e) {
+    return false;
+  }
+}
+
 export {
   InAppMessage,
   InAppMessageClickEvent,