feat: add marketing consent direct login options

Author: CodeSchwert

src/Packages/Passport/Runtime/Scripts/Private/Model/DirectLoginMethod.cs

@@ -8,6 +8,7 @@ namespace Immutable.Passport.Model
     [Serializable]
     public enum DirectLoginMethod
     {
+        None,
         Email,
         Google,
         Apple,

src/Packages/Passport/Runtime/Scripts/Private/Model/DirectLoginOptions.cs

@@ -19,24 +19,32 @@ public class DirectLoginOptions
         /// </summary>
         public string email;
 
+        /// <summary>
+        /// Marketing consent status (optional).
+        /// </summary>
+        public MarketingConsentStatus? marketingConsentStatus;
+
         /// <summary>
         /// Default constructor.
         /// </summary>
         public DirectLoginOptions()
         {
             directLoginMethod = DirectLoginMethod.Email;
             email = null;
+            marketingConsentStatus = null;
         }
 
         /// <summary>
         /// Constructor with method and email.
         /// </summary>
         /// <param name="loginMethod">The direct login method</param>
         /// <param name="emailAddress">The email address (optional)</param>
-        public DirectLoginOptions(DirectLoginMethod loginMethod, string emailAddress = null)
+        /// <param name="marketingConsentStatus">The marketing consent status (optional)</param>
+        public DirectLoginOptions(DirectLoginMethod loginMethod, string emailAddress = null, MarketingConsentStatus? marketingConsentStatus = null)
         {
             directLoginMethod = loginMethod;
             email = emailAddress;
+            this.marketingConsentStatus = marketingConsentStatus;
         }
 
         /// <summary>

src/Packages/Passport/Runtime/Scripts/Private/Model/MarketingConsentStatus.cs

@@ -0,0 +1,35 @@
+using System;
+
+namespace Immutable.Passport.Model
+{
+    /// <summary>
+    /// Enum representing marketing consent status.
+    /// </summary>
+    [Serializable]
+    public enum MarketingConsentStatus
+    {
+        OptedIn,
+        Unsubscribed
+    }
+
+    /// <summary>
+    /// Extension methods for MarketingConsentStatus enum.
+    /// </summary>
+    public static class MarketingConsentStatusExtensions
+    {
+        /// <summary>
+        /// Converts the enum value to the string format expected by the game bridge.
+        /// </summary>
+        /// <param name="status">The marketing consent status</param>
+        /// <returns>The corresponding string value</returns>
+        public static string ToApiString(this MarketingConsentStatus status)
+        {
+            return status switch
+            {
+                MarketingConsentStatus.OptedIn => "opted_in",
+                MarketingConsentStatus.Unsubscribed => "unsubscribed",
+                _ => throw new ArgumentOutOfRangeException(nameof(status), status, "Unknown MarketingConsentStatus value")
+            };
+        }
+    }
+}

src/Packages/Passport/Runtime/Scripts/Private/Model/MarketingConsentStatus.cs.meta

@@ -275,22 +275,19 @@ private async UniTask LaunchAuthUrl()
         {
             try
             {
-                // Create the request JSON manually to ensure proper serialization
-                var requestJson = $"{{\"isConnectImx\":{(!_pkceLoginOnly).ToString().ToLower()}";
-
-                if (_directLoginOptions != null)
+                // Create the request using a serializable class for clean JSON generation
+                var request = new AuthUrlRequest
                 {
-                    requestJson += $",\"directLoginOptions\":{{\"directLoginMethod\":\"{_directLoginOptions.directLoginMethod.ToString().ToLower()}\"";
-
-                    if (_directLoginOptions.IsEmailValid())
+                    isConnectImx = !_pkceLoginOnly,
+                    directLoginOptions = _directLoginOptions != null ? new DirectLoginRequestOptions
                     {
-                        requestJson += $",\"email\":\"{_directLoginOptions.email}\"";
-                    }
+                        directLoginMethod = _directLoginOptions.directLoginMethod.ToString().ToLower(),
+                        email = _directLoginOptions.IsEmailValid() ? _directLoginOptions.email : null,
+                        marketingConsentStatus = _directLoginOptions.marketingConsentStatus?.ToApiString()
+                    } : null
+                };
 
-                    requestJson += "}";
-                }
-
-                requestJson += "}";
+                var requestJson = JsonUtility.ToJson(request);
 
                 var callResponse = await _communicationsManager.Call(PassportFunction.GET_PKCE_AUTH_URL, requestJson);
                 var response = callResponse.OptDeserializeObject<StringResponse>();
@@ -299,10 +296,6 @@ private async UniTask LaunchAuthUrl()
                 {
                     var url = response.result.Replace(" ", "+");
 
-                    // force marketing consent to true for now
-                    // TODO: remove this once we have a way to get the marketing consent from the user
-                    url = url + "&marketingConsent=opted_in";
-
 #if UNITY_ANDROID && !UNITY_EDITOR
                     loginPKCEUrl = url;
                     SendAuthEvent(_pkceLoginOnly ? PassportAuthEvent.LoginPKCELaunchingCustomTabs : PassportAuthEvent.ConnectImxPKCELaunchingCustomTabs);
@@ -818,4 +811,25 @@ async void onDeeplinkResult(string url)
         }
     }
 #endif
+
+    /// <summary>
+    /// Serializable request class for LaunchAuthUrl to replace manual JSON string concatenation
+    /// </summary>
+    [Serializable]
+    internal class AuthUrlRequest
+    {
+        public bool isConnectImx;
+        public DirectLoginRequestOptions directLoginOptions;
+    }
+
+    /// <summary>
+    /// Serializable class for directLoginOptions within AuthUrlRequest
+    /// </summary>
+    [Serializable]
+    internal class DirectLoginRequestOptions
+    {
+        public string directLoginMethod;
+        public string email;
+        public string marketingConsentStatus;
+    }
 }

src/Packages/Passport/Runtime/Scripts/Private/PassportImpl.cs

@@ -449,6 +449,23 @@ private async void HandleLoginData(string jsonData)
                     loginOptions.email = loginData.email;
                 }
 
+                // Parse and set marketing consent status if provided
+                if (!string.IsNullOrEmpty(loginData.marketingConsentStatus))
+                {
+                    switch (loginData.marketingConsentStatus.ToLower())
+                    {
+                        case "opted_in":
+                            loginOptions.marketingConsentStatus = MarketingConsentStatus.OptedIn;
+                            break;
+                        case "unsubscribed":
+                            loginOptions.marketingConsentStatus = MarketingConsentStatus.Unsubscribed;
+                            break;
+                        default:
+                            PassportLogger.Warn($"{TAG} Unknown marketing consent status: {loginData.marketingConsentStatus}");
+                            break;
+                    }
+                }
+
                 // Perform the login and handle the result
                 PassportLogger.Info($"{TAG} Starting login with method: {loginData.directLoginMethod}");
                 bool loginSuccess = await _passportInstance.Login(false, loginOptions);