Add InitializeOnStart option to ThirdwebManager

Introduces a new InitializeOnStart serialized property to ThirdwebManagerBase and its editor, allowing initialization to occur in Start() instead of Awake().

Author: 0xFirekeeper

Assets/Thirdweb/Editor/ThirdwebManagerEditor.cs

@@ -10,6 +10,7 @@ public abstract class ThirdwebManagerBaseEditor<T> : UnityEditor.Editor
         where T : MonoBehaviour
     {
         protected SerializedProperty InitializeOnAwakeProp;
+        protected SerializedProperty InitializeOnStartProp;
         protected SerializedProperty ShowDebugLogsProp;
         protected SerializedProperty AutoConnectLastWalletProp;
         protected SerializedProperty RedirectPageHtmlOverrideProp;
@@ -24,6 +25,7 @@ public abstract class ThirdwebManagerBaseEditor<T> : UnityEditor.Editor
         protected virtual void OnEnable()
         {
             this.InitializeOnAwakeProp = this.FindProp("InitializeOnAwake");
+            this.InitializeOnStartProp = this.FindProp("InitializeOnStart");
             this.ShowDebugLogsProp = this.FindProp("ShowDebugLogs");
             this.AutoConnectLastWalletProp = this.FindProp("AutoConnectLastWallet");
             this.RedirectPageHtmlOverrideProp = this.FindProp("RedirectPageHtmlOverride");
@@ -77,6 +79,7 @@ protected virtual void DrawPreferencesTab()
         {
             EditorGUILayout.HelpBox("Set your preferences and initialization options here.", MessageType.Info);
             this.DrawProperty(this.InitializeOnAwakeProp, "Initialize On Awake");
+            this.DrawProperty(this.InitializeOnStartProp, "Initialize On Start");
             this.DrawProperty(this.ShowDebugLogsProp, "Show Debug Logs");
             this.DrawProperty(this.AutoConnectLastWalletProp, "Auto-Connect Last Wallet");
         }

Assets/Thirdweb/Runtime/Unity/ThirdwebManager.cs

@@ -12,40 +12,37 @@ public class ThirdwebManager : ThirdwebManagerBase
         [field: SerializeField]
         private string BundleId { get; set; }
 
-        public static new ThirdwebManager Instance
-        {
-            get => ThirdwebManagerBase.Instance as ThirdwebManager;
-        }
+        public static new ThirdwebManager Instance => ThirdwebManagerBase.Instance as ThirdwebManager;
 
         protected override ThirdwebClient CreateClient()
         {
-            if (string.IsNullOrWhiteSpace(ClientId))
+            if (string.IsNullOrWhiteSpace(this.ClientId))
             {
                 ThirdwebDebug.LogError("ClientId must be set in order to initialize ThirdwebManager. " + "Get your API key from https://thirdweb.com/create-api-key");
                 return null;
             }
 
-            if (string.IsNullOrWhiteSpace(BundleId))
+            if (string.IsNullOrWhiteSpace(this.BundleId))
             {
-                BundleId = null;
+                this.BundleId = null;
             }
 
-            BundleId ??= string.IsNullOrWhiteSpace(Application.identifier) ? $"com.{Application.companyName}.{Application.productName}" : Application.identifier;
+            this.BundleId ??= string.IsNullOrWhiteSpace(Application.identifier) ? $"com.{Application.companyName}.{Application.productName}" : Application.identifier;
 
             return ThirdwebClient.Create(
-                clientId: ClientId,
-                bundleId: BundleId,
+                clientId: this.ClientId,
+                bundleId: this.BundleId,
                 httpClient: new CrossPlatformUnityHttpClient(),
                 sdkName: Application.platform == RuntimePlatform.WebGLPlayer ? "UnitySDK_WebGL" : "UnitySDK",
                 sdkOs: Application.platform.ToString(),
                 sdkPlatform: "unity",
                 sdkVersion: THIRDWEB_UNITY_SDK_VERSION,
-                rpcOverrides: (RpcOverrides == null || RpcOverrides.Count == 0)
+                rpcOverrides: (this.RpcOverrides == null || this.RpcOverrides.Count == 0)
                     ? null
-                    : RpcOverrides.ToDictionary(rpcOverride => new BigInteger(rpcOverride.ChainId), rpcOverride => rpcOverride.RpcUrl)
+                    : this.RpcOverrides.ToDictionary(rpcOverride => new BigInteger(rpcOverride.ChainId), rpcOverride => rpcOverride.RpcUrl)
             );
         }
 
-        public override string MobileRedirectScheme => BundleId + "://";
+        public override string MobileRedirectScheme => this.BundleId + "://";
     }
 }

Assets/Thirdweb/Runtime/Unity/ThirdwebManagerBase.cs

@@ -266,6 +266,9 @@ public abstract class ThirdwebManagerBase : MonoBehaviour
         [field: SerializeField]
         protected bool InitializeOnAwake { get; set; } = true;
 
+        [field: SerializeField]
+        protected bool InitializeOnStart { get; set; } = false;
+
         [field: SerializeField]
         protected bool ShowDebugLogs { get; set; } = true;
 
@@ -339,8 +342,22 @@ protected virtual void Awake()
             }
         }
 
+        protected virtual void Start()
+        {
+            if (this.InitializeOnStart)
+            {
+                this.Initialize();
+            }
+        }
+
         public virtual async void Initialize()
         {
+            if (this.Initialized || this.Client != null)
+            {
+                ThirdwebDebug.LogWarning("ThirdwebManager is already initialized.");
+                return;
+            }
+
             this.Client = this.CreateClient();
             if (this.Client == null)
             {