chore: update screen tracking parameters

android/src/main/kotlin/io/customer/customer_io/CustomerIoPlugin.kt

@@ -22,6 +22,7 @@ import io.flutter.embedding.engine.plugins.activity.ActivityPluginBinding
 import io.flutter.plugin.common.MethodCall
 import io.flutter.plugin.common.MethodChannel
 import io.flutter.plugin.common.MethodChannel.MethodCallHandler
+import io.flutter.plugin.common.MethodChannel.Result
 import java.lang.ref.WeakReference
 
 /**
@@ -171,9 +172,11 @@ class CustomerIoPlugin : FlutterPlugin, MethodCallHandler, ActivityAware {
 
     private fun track(params: Map<String, Any>) {
         val name = requireNotNull(params.getAsTypeOrNull<String>(Keys.Tracking.NAME)) {
-            "Event name is required to track event"
+            "Event name is missing in params: $params"
         }
-        val properties = params.getAsTypeOrNull<Map<String, Any>>(Keys.Tracking.PROPERTIES)
+        val properties = params
+            .getAsTypeOrNull<Map<String, Any?>>(Keys.Tracking.PROPERTIES)
+            ?.filterValues { it != null }
 
         if (properties.isNullOrEmpty()) {
             CustomerIO.instance().track(name)
@@ -225,18 +228,18 @@ class CustomerIoPlugin : FlutterPlugin, MethodCallHandler, ActivityAware {
     }
 
     private fun screen(params: Map<String, Any>) {
-        // TODO: Fix screen implementation
-        /*
-        val name = params.getString(Keys.Tracking.EVENT_NAME)
-        val attributes =
-            params.getProperty<Map<String, Any>>(Keys.Tracking.ATTRIBUTES) ?: emptyMap()
+        val title = requireNotNull(params.getAsTypeOrNull<String>(Keys.Tracking.TITLE)) {
+            "Screen title is missing in params: $params"
+        }
+        val properties = params
+            .getAsTypeOrNull<Map<String, Any?>>(Keys.Tracking.PROPERTIES)
+            ?.filterValues { it != null }
 
-        if (attributes.isEmpty()) {
-            CustomerIO.instance().screen(name)
+        if (properties.isNullOrEmpty()) {
+            CustomerIO.instance().screen(title)
         } else {
-            CustomerIO.instance().screen(name, attributes)
+            CustomerIO.instance().screen(title, properties)
         }
-         */
     }
 
     private fun initialize(args: Map<String, Any>): kotlin.Result<Unit> = runCatching {

android/src/main/kotlin/io/customer/customer_io/constant/Keys.kt

@@ -28,5 +28,6 @@ internal object Keys {
 
         const val NAME = "name"
         const val PROPERTIES = "properties"
+        const val TITLE = "title"
     }
 }

apps/amiapp_flutter/lib/src/app.dart

@@ -216,7 +216,7 @@ class _AmiAppState extends State<AmiApp> {
     if (_customerIOSDK.sdkConfig?.screenTrackingEnabled == true) {
       final Screen? screen = _router.currentLocation().toAppScreen();
       if (screen != null) {
-        CustomerIO.instance.screen(name: screen.name);
+        CustomerIO.instance.screen(title: screen.name);
       }
     }
   }

apps/amiapp_flutter/lib/src/data/config.dart

@@ -8,8 +8,8 @@ class CustomerIOSDKConfig {
   final String cdpApiKey;
   final String? migrationSiteId;
   final Region? region;
-  final bool? debugModeEnabled;
-  final bool? screenTrackingEnabled;
+  final bool debugModeEnabled;
+  final bool screenTrackingEnabled;
   final bool? autoTrackDeviceAttributes;
   final String? apiHost;
   final String? cdnHost;
@@ -22,8 +22,8 @@ class CustomerIOSDKConfig {
     required this.cdpApiKey,
     this.migrationSiteId,
     this.region,
-    this.debugModeEnabled,
-    this.screenTrackingEnabled,
+    this.debugModeEnabled = true,
+    this.screenTrackingEnabled = true,
     this.autoTrackDeviceAttributes,
     this.apiHost,
     this.cdnHost,
@@ -53,9 +53,10 @@ class CustomerIOSDKConfig {
       cdpApiKey: cdpApiKey,
       migrationSiteId: prefs.getString(_PreferencesKey.migrationSiteId),
       region: region,
-      debugModeEnabled: prefs.getBool(_PreferencesKey.debugModeEnabled),
+      debugModeEnabled:
+          prefs.getBool(_PreferencesKey.debugModeEnabled) != false,
       screenTrackingEnabled:
-          prefs.getBool(_PreferencesKey.screenTrackingEnabled),
+          prefs.getBool(_PreferencesKey.screenTrackingEnabled) != false,
       autoTrackDeviceAttributes:
           prefs.getBool(_PreferencesKey.autoTrackDeviceAttributes),
       apiHost: prefs.getString(_PreferencesKey.apiHost),

ios/Classes/Keys.swift

@@ -27,6 +27,7 @@ struct Keys {
 
         static let name = "name"
         static let properties = "properties"
+        static let title = "title"
     }
 
     struct Environment{

ios/Classes/SwiftCustomerIoPlugin.swift

@@ -106,20 +106,17 @@ public class SwiftCustomerIoPlugin: NSObject, FlutterPlugin {
     }
 
     func screen(params : Dictionary<String, AnyHashable>) {
-        // TODO: Fix screen implementation
-        /*
-         guard let name = params[Keys.Tracking.eventName] as? String
-         else {
-         return
-         }
-
-         guard let attributes = params[Keys.Tracking.attributes] as? Dictionary<String, AnyHashable> else{
-         CustomerIO.shared.screen(name: name)
-         return
-         }
-
-         CustomerIO.shared.screen(name: name, data: attributes)
-         */
+        guard let title = params[Keys.Tracking.title] as? String else {
+            logger.error("Missing screen title in: \(params) for key: \(Keys.Tracking.title)")
+            return
+        }
+
+        guard let properties = params[Keys.Tracking.properties] as? Dictionary<String, AnyHashable> else {
+            CustomerIO.shared.screen(title: title)
+            return
+        }
+
+        CustomerIO.shared.screen(title: title, properties: properties)
     }
 
 

lib/customer_io.dart

@@ -131,8 +131,8 @@ class CustomerIO {
   /// @param name name of the screen user visited
   /// @param attributes (Optional) params to be sent with event
   void screen(
-      {required String name, Map<String, dynamic> attributes = const {}}) {
-    return _platform.screen(name: name, attributes: attributes);
+      {required String title, Map<String, dynamic> properties = const {}}) {
+    return _platform.screen(title: title, properties: properties);
   }
 
   /// Use this function to send custom device attributes

lib/customer_io_const.dart

@@ -28,4 +28,5 @@ class TrackingConsts {
 
   static const String name = "name";
   static const String properties = "properties";
+  static const String title = "title";
 }

lib/customer_io_method_channel.dart

@@ -128,12 +128,12 @@ class CustomerIOMethodChannel extends CustomerIOPlatform {
   /// Track screen events to record the screens a user visits
   @override
   void screen(
-      {required String name,
-      Map<String, dynamic> attributes = const {}}) async {
+      {required String title,
+      Map<String, dynamic> properties = const {}}) async {
     try {
       final payload = {
-        TrackingConsts.eventName: name,
-        TrackingConsts.attributes: attributes
+        TrackingConsts.title: title,
+        TrackingConsts.properties: properties
       };
       methodChannel.invokeMethod(MethodConsts.screen, payload);
     } on PlatformException catch (exception) {

lib/customer_io_platform_interface.dart

@@ -61,7 +61,7 @@ abstract class CustomerIOPlatform extends PlatformInterface {
   }
 
   void screen(
-      {required String name, Map<String, dynamic> attributes = const {}}) {
+      {required String title, Map<String, dynamic> properties = const {}}) {
     throw UnimplementedError('screen() has not been implemented.');
   }
 

test/customer_io_method_channel_test.dart

@@ -105,12 +105,12 @@ void main() {
 
   test('screen() should call platform method with correct arguments', () async {
     final Map<String, dynamic> args = {
-      'eventName': 'screen_event',
-      'attributes': {'screenName': '你好'}
+      'title': 'screen_event',
+      'properties': {'screenName': '你好'}
     };
 
     final customerIO = CustomerIOMethodChannel();
-    customerIO.screen(name: args['eventName'], attributes: args['attributes']);
+    customerIO.screen(title: args['title'], properties: args['properties']);
 
     expectMethodInvocationArguments('screen', args);
   });

test/customer_io_test.dart

@@ -149,6 +149,19 @@ void main() {
         );
       });
 
+      test('screen() correct arguments are passed', () {
+        const title = 'checkout';
+        final givenProperties = {'source': 'push'};
+        CustomerIO.instance.screen(title: title, properties: givenProperties);
+        expect(
+          verify(mockPlatform.screen(
+            title: captureAnyNamed("title"),
+            properties: captureAnyNamed("properties"),
+          )).captured,
+          [title, givenProperties],
+        );
+      });
+
       test('trackMetric() calls platform', () {
         const deliveryID = '123';
         const deviceToken = 'abc';

test/customer_io_test.mocks.dart

@@ -124,16 +124,16 @@ class MockTestCustomerIoPlatform extends _i1.Mock
       );
   @override
   void screen({
-    required String? name,
-    Map<String, dynamic>? attributes = const {},
+    required String? title,
+    Map<String, dynamic>? properties = const {},
   }) =>
       super.noSuchMethod(
         Invocation.method(
           #screen,
           [],
           {
-            #name: name,
-            #attributes: attributes,
+            #title: title,
+            #properties: properties,
           },
         ),
         returnValueForMissingStub: null,