fixing mobile login with Awaitable

Runtime/Scripts/Beacon/BeaconConnectorDotNet.cs

@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;
+using System.Threading.Tasks;
 using Beacon.Sdk;
 using Beacon.Sdk.Beacon;
 using Beacon.Sdk.Beacon.Operation;
@@ -32,7 +33,7 @@ public class BeaconConnectorDotNet : IBeaconConnector, IDisposable
 
         #region IBeaconConnector
 
-        public async void ConnectAccount()
+        public async Task ConnectAccount()
         {
             if (BeaconDappClient != null) return;
 
@@ -102,8 +103,7 @@ public void SetWalletMessageReceiver(WalletMessageReceiver messageReceiver)
         {
             _walletMessageReceiver = messageReceiver;
         }
-
-        public async void RequestTezosPermission(string networkName = "", string networkRPC = "")
+        public async Task RequestTezosPermission(string networkName = "", string networkRPC = "")
         {
             if (!Enum.TryParse(networkName, out NetworkType networkType))
                 networkType = TezosConfig.Instance.Network;

Runtime/Scripts/Beacon/IBeaconConnector.cs

@@ -1,6 +1,7 @@
+using System.Threading.Tasks;
 using Beacon.Sdk.Beacon.Sign;
-using Netezos.Forging.Models;
 using TezosSDK.Tezos.Wallet;
+using UnityEngine;
 
 namespace TezosSDK.Beacon
 {
@@ -27,7 +28,7 @@ public interface IBeaconConnector
 		/// Starts the connection between Beacon SDK and a wallet to connect to
 		/// an account
 		/// </summary>
-		void ConnectAccount();
+		Task ConnectAccount();
 
 		/// <summary>
 		/// Checks if there is an active account paired.
@@ -40,7 +41,7 @@ public interface IBeaconConnector
 		/// </summary>
 		/// <param name="networkName">The name of the desired network.</param>
 		/// <param name="networkRPC">The RPC to the desired network</param>
-		public void RequestTezosPermission(string networkName = "", string networkRPC = "");
+		public Task RequestTezosPermission(string networkName = "", string networkRPC = "");
 
 		/// <summary>
 		/// Allows requesting a new operation such as sending tezos or initiating a smart contract.

Runtime/Scripts/DesignPattern/Singleton/SingletonMonoBehaviour.cs

@@ -54,7 +54,7 @@ public static T Instantiate()
         {
             if (!IsInstantiated())
             {
-                T t = GameObject.FindObjectOfType<T>();
+                T t = FindFirstObjectByType<T>();
                 GameObject go = null;
                 if (t != null)
                 {

Runtime/Scripts/Helpers/TaskExtension.cs

@@ -0,0 +1,15 @@
+using System.Collections;
+using System.Threading.Tasks;
+using UnityEngine;
+
+namespace TezosSDK.Helpers
+{
+    public static class TaskExtension
+    {
+        public static IEnumerator ToCoroutine(this Task task)
+        {
+            var t = Task.Run(async () => await task);
+            yield return new WaitUntil(() => t.IsCompleted);
+        }
+    }
+}

Runtime/Scripts/Tezos/TezosConfig.cs

@@ -11,7 +11,7 @@ public class TezosConfig
 
         public NetworkType Network { get; set; } = NetworkType.ghostnet;
 
-        public string RpcBaseUrl => $"https://rpc.{Network}.teztnets.xyz";
+        public string RpcBaseUrl => $"https://{Network}.tezos.marigold.dev";
         public int DefaultTimeoutSeconds => 45;
     }
 

Runtime/Scripts/Tezos/Wallet/IWalletProvider.cs

@@ -1,5 +1,8 @@
-using Beacon.Sdk.Beacon.Sign;
+using System.Collections;
+using System.Threading.Tasks;
+using Beacon.Sdk.Beacon.Sign;
 using TezosSDK.Beacon;
+using UnityEngine;
 
 namespace TezosSDK.Tezos.Wallet
 {
@@ -19,7 +22,7 @@ public interface IWalletProvider
         /// Makes a call to connect with a wallet
         /// <param name="withRedirectToWallet">Should we open wallet app on mobiles after connect?</param>
         /// </summary>
-        void Connect(WalletProviderType walletProvider, bool withRedirectToWallet = true);
+        IEnumerator Connect(WalletProviderType walletProvider, bool withRedirectToWallet = true);
 
         /// <summary>
         /// Unpair with wallet and disconnect

Runtime/Scripts/Tezos/Wallet/WalletProvider.cs

@@ -1,6 +1,8 @@
 using System;
 using System.Collections;
+using System.Runtime.CompilerServices;
 using System.Text.Json;
+using System.Threading.Tasks;
 using Beacon.Sdk.Beacon.Sign;
 using TezosSDK.Beacon;
 using TezosSDK.Helpers;
@@ -20,10 +22,10 @@ public class WalletProvider : IWalletProvider, IDisposable
 
         public WalletProvider()
         {
-            InitBeaconConnector();
+            CoroutineRunner.Instance.StartWrappedCoroutine(InitBeaconConnector());
         }
 
-        private void InitBeaconConnector()
+        private IEnumerator InitBeaconConnector()
         {
             // Create or get a WalletMessageReceiver Game object to receive callback messages
             var unityBeacon = GameObject.Find("UnityBeacon");
@@ -37,7 +39,7 @@ private void InitBeaconConnector()
 #else
             _beaconConnector = new BeaconConnectorDotNet();
             (_beaconConnector as BeaconConnectorDotNet)?.SetWalletMessageReceiver(MessageReceiver);
-            Connect(WalletProviderType.beacon, withRedirectToWallet: false);
+            yield return Connect(WalletProviderType.beacon, withRedirectToWallet: false);
 
             // todo: maybe call RequestTezosPermission from _beaconConnector?
             MessageReceiver.PairingCompleted += _ =>
@@ -47,7 +49,11 @@ private void InitBeaconConnector()
                     networkRPC: TezosConfig.Instance.RpcBaseUrl);
             };
 #endif
-            MessageReceiver.HandshakeReceived += handshake => { _handshake = handshake; };
+            MessageReceiver.HandshakeReceived += handshake =>
+            {
+                Debug.Log($"Handshake Received: ${handshake}");
+                _handshake = handshake;
+            };
             MessageReceiver.AccountConnected += account =>
             {
                 var json = JsonSerializer.Deserialize<JsonElement>(account);
@@ -72,43 +78,41 @@ private void InitBeaconConnector()
             };
         }
 
-        // Below there are some async/wait workarounds and magic numbers, 
-        // we should rewrite the Beacon connector to be coroutine compatible.
-        private IEnumerator OnOpenWallet(bool withRedirectToWallet)
-        {
-            if (string.IsNullOrEmpty(_handshake))
-            {
-                //No handshake, Waiting for handshake...
-                yield return new WaitForSeconds(2.5f);
-            }
-
-#if UNITY_ANDROID || UNITY_IOS
-            if (withRedirectToWallet){
-                _beaconConnector.RequestTezosPermission(
-                    networkName: TezosConfig.Instance.Network.ToString(),
-                    networkRPC: TezosConfig.Instance.RpcBaseUrl);
-                yield return new WaitForSeconds(2.5f);
-                if (!string.IsNullOrEmpty(_handshake)){
-                    Application.OpenURL($"tezos://?type=tzip10&data={_handshake}");
-                }
-            }
-#endif
-        }
-
         public void OnReady()
         {
             _beaconConnector.OnReady();
         }
 
-        public void Connect(WalletProviderType walletProvider, bool withRedirectToWallet)
+        public IEnumerator Connect(WalletProviderType walletProvider, bool withRedirectToWallet)
         {
-
+            Debug.Log("InitWalletProvider");
             _beaconConnector.InitWalletProvider(
                 network: TezosConfig.Instance.Network.ToString(),
                 rpc: TezosConfig.Instance.RpcBaseUrl,
                 walletProviderType: walletProvider);
-            _beaconConnector.ConnectAccount();
-            CoroutineRunner.Instance.StartWrappedCoroutine(OnOpenWallet(withRedirectToWallet));
+            Debug.Log("ConnectAccount");
+            var connectAccountCoroutine = _beaconConnector.ConnectAccount().ToCoroutine();
+            yield return connectAccountCoroutine;
+
+#if UNITY_ANDROID || UNITY_IOS
+            if (withRedirectToWallet){
+                Debug.Log("RequestTezosPermission");
+                yield return _beaconConnector.RequestTezosPermission(
+                    networkName: TezosConfig.Instance.Network.ToString(),
+                    networkRPC: TezosConfig.Instance.RpcBaseUrl).ToCoroutine();
+                if (string.IsNullOrEmpty(_handshake))
+                {
+                    //No handshake, Waiting for handshake...
+                    Debug.Log("No handshake, Waiting for handshake...");
+                    // yield return new WaitUntil(() => string.IsNullOrEmpty(_handshake));
+                    yield return new WaitForSecondsRealtime(2.5f);
+                }
+                if (!string.IsNullOrEmpty(_handshake)){
+                    Debug.Log("tezos://?type=tzip10&data=" + _handshake);
+                    Application.OpenURL($"tezos://?type=tzip10&data={_handshake}");
+                }
+            }
+#endif
         }
 
         public void Disconnect()