Update tagging methods to pass string value to native bridge

Author: jennantilla

ios/RCTOneSignal/RCTOneSignalEventEmitter.m

@@ -353,14 +353,9 @@ + (void)sendEventWithName:(NSString *)name withBody:(NSDictionary *)body {
 }
 
 RCT_EXPORT_METHOD(addTag:(NSString *)key value:(id)value) {
- if([value isKindOfClass:[NSNumber class]]) {
-         //It is a number, convert to a string
-         value = [value stringValue];
- }
- [OneSignal.User addTagWithKey:key value:value];
+    [OneSignal.User addTagWithKey:key value:value];
 }
 
-
 RCT_EXPORT_METHOD(addTags:(NSDictionary *)tags) {
     [OneSignal.User addTags:tags];
 }

src/index.ts

@@ -430,15 +430,20 @@ export namespace OneSignal {
      * Add a tag for the current user. Tags are key:value pairs used as building blocks for targeting specific users and/or personalizing
      * messages. If the tag key already exists, it will be replaced with the value provided here.
      */
-    export function addTag(key: string, value: string | number | boolean) {
+    export function addTag(key: string, value: string) {
       if (!isNativeModuleLoaded(RNOneSignal)) return;
 
-      if (!key || (!value && value !== '')) {
+      if (!key || !value) {
         console.error('OneSignal: sendTag: must include a key and a value');
         return;
       }
 
-      RNOneSignal.addTag(key, value.toString());
+      // forces values to be string types
+      if (typeof value !== 'string') {
+        value = String(value);
+      }
+
+      RNOneSignal.addTag(key, value);
     }
 
     /**
@@ -456,6 +461,14 @@ export namespace OneSignal {
         return;
       }
 
+      const convertedTags = tags as { [key: string]: any };
+      Object.keys(tags).forEach(function (key) {
+        // forces values to be string types
+        if (typeof convertedTags[key] !== 'string') {
+          convertedTags[key] = JSON.stringify(convertedTags[key]);
+        }
+      });
+
       RNOneSignal.addTags(tags);
     }