@react-native-firebase/analytics 14.2.2 → 14.2.3

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [14.2.3](https://github.com/invertase/react-native-firebase/compare/v14.2.2...v14.2.3) (2022-01-20)
+
+### Bug Fixes
+
+- **analytics, ios:** Convert NSNull values to nil in order to correctly remove user properties ([dbc79c3](https://github.com/invertase/react-native-firebase/commit/dbc79c31810bf9bea4c3a782432c9b5625ad3d5f)), closes [#4931](https://github.com/invertase/react-native-firebase/issues/4931)
+
 ## [14.2.2](https://github.com/invertase/react-native-firebase/compare/v14.2.1...v14.2.2) (2022-01-06)
 
 **Note:** Version bump only for package @react-native-firebase/analytics

package/ios/RNFBAnalytics/RNFBAnalyticsModule.m

@@ -66,7 +66,7 @@ RCT_EXPORT_METHOD(setUserId
                   : (RCTPromiseResolveBlock)resolve rejecter
                   : (RCTPromiseRejectBlock)reject) {
   @try {
-    [FIRAnalytics setUserID:id];
+    [FIRAnalytics setUserID:[self convertNSNullToNil:id]];
   } @catch (NSException *exception) {
     return [RNFBSharedUtils rejectPromiseWithExceptionDict:reject exception:exception];
   }
@@ -79,7 +79,7 @@ RCT_EXPORT_METHOD(setUserProperty
                   : (RCTPromiseResolveBlock)resolve rejecter
                   : (RCTPromiseRejectBlock)reject) {
   @try {
-    [FIRAnalytics setUserPropertyString:value forName:name];
+    [FIRAnalytics setUserPropertyString:[self convertNSNullToNil:value] forName:name];
   } @catch (NSException *exception) {
     return [RNFBSharedUtils rejectPromiseWithExceptionDict:reject exception:exception];
   }
@@ -92,7 +92,7 @@ RCT_EXPORT_METHOD(setUserProperties
                   : (RCTPromiseRejectBlock)reject) {
   @try {
     [properties enumerateKeysAndObjectsUsingBlock:^(id key, id value, BOOL *stop) {
-      [FIRAnalytics setUserPropertyString:value forName:key];
+      [FIRAnalytics setUserPropertyString:[self convertNSNullToNil:value] forName:key];
     }];
   } @catch (NSException *exception) {
     return [RNFBSharedUtils rejectPromiseWithExceptionDict:reject exception:exception];
@@ -162,4 +162,10 @@ RCT_EXPORT_METHOD(setDefaultEventParameters
   return [newParams copy];
 }
 
+/// Converts null values received over the bridge from NSNull to nil
+/// @param value Nullable string value
+- (NSString *)convertNSNullToNil:(NSString *)value {
+  return [value isEqual:[NSNull null]] ? nil : value;
+}
+
 @end

package/lib/version.js

@@ -1,2 +1,2 @@
 // generated by genversion
-module.exports = '14.2.2';
+module.exports = '14.2.3';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-firebase/analytics",
-  "version": "14.2.2",
+  "version": "14.2.3",
   "author": "Invertase <oss@invertase.io> (http://invertase.io)",
   "description": "React Native Firebase - The analytics module provides out of the box support with Google Analytics for Firebase. Integration with the Android & iOS allows for in-depth analytical insight reporting, such as device information, location, user actions and more.",
   "main": "lib/index.js",
@@ -22,10 +22,10 @@
     "analytics"
   ],
   "peerDependencies": {
-    "@react-native-firebase/app": "14.2.2"
+    "@react-native-firebase/app": "14.2.3"
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "7b0de31e45be9ca7571583e77d1b06c3088df053"
+  "gitHead": "eb58323969bc7803b0695bd90cdb7dc3436179e1"
 }