adding popups to connect status message pattern

Author: fernando-cortez

Assets/BossRoom/Scripts/Client/UI/ConnectionStatusMessageUIManager.cs

@@ -51,6 +51,13 @@ void OnConnectStatus(ConnectStatus status)
                 case ConnectStatus.HostEndedSession:
                     PopupManager.ShowPopupPanel("Disconnected From Host", "The host has ended the game session.");
                     break;
+                case ConnectStatus.StartHostFailed:
+                    PopupManager.ShowPopupPanel("Connection Failed", "Server failed to bind.");
+                    break;
+                case ConnectStatus.StartClientFailed:
+                    PopupManager.ShowPopupPanel("Connection Failed",
+                        "Failed to connect to server and/or invalid network endpoint.");
+                    break;
                 default:
                     Debug.LogWarning($"New ConnectStatus {status} has been added, but no connect message defined for it.");
                     break;

Assets/BossRoom/Scripts/Client/UI/IPConnectionWindow.cs

@@ -19,10 +19,13 @@ public class IPConnectionWindow : MonoBehaviour
 
         IPUIMediator m_IPUIMediator;
 
+        IPublisher<ConnectStatus> m_ConnectStatusPublisher;
+
         [Inject]
-        void InjectDependencies(IPUIMediator ipUIMediator)
+        void InjectDependencies(IPUIMediator ipUIMediator, IPublisher<ConnectStatus> connectStatusPublisher)
         {
             m_IPUIMediator = ipUIMediator;
+            m_ConnectStatusPublisher = connectStatusPublisher;
         }
 
         void Awake()
@@ -49,8 +52,7 @@ public void ShowConnectingWindow()
 
             void OnTimeElapsed()
             {
-                PopupManager.ShowPopupPanel("Connection Failed",
-                    "Failed to connect to server and/or invalid network endpoint");
+                m_ConnectStatusPublisher.Publish(ConnectStatus.StartClientFailed);
                 Hide();
                 m_IPUIMediator.DisableSignInSpinner();
             }

Assets/BossRoom/Scripts/Client/UI/IPUIMediator.cs

@@ -37,17 +37,20 @@ public class IPUIMediator : MonoBehaviour
         NameGenerationData m_NameGenerationData;
         GameNetPortal m_GameNetPortal;
         ClientGameNetPortal m_ClientNetPortal;
+        IPublisher<ConnectStatus> m_ConnectStatusPublisher;
 
         [Inject]
         void InjectDependenciesAndInitialize(
             NameGenerationData nameGenerationData,
             GameNetPortal gameNetPortal,
-            ClientGameNetPortal clientGameNetPortal
+            ClientGameNetPortal clientGameNetPortal,
+            IPublisher<ConnectStatus> connectStatusPublisher
         )
         {
             m_NameGenerationData = nameGenerationData;
             m_GameNetPortal = gameNetPortal;
             m_ClientNetPortal = clientGameNetPortal;
+            m_ConnectStatusPublisher = connectStatusPublisher;
 
             RegenerateName();
         }
@@ -81,7 +84,7 @@ public void HostIPRequest(string ip, string port)
             }
             else
             {
-                PopupManager.ShowPopupPanel("Connection Failed", "Server failed to bind.");
+                m_ConnectStatusPublisher.Publish(ConnectStatus.StartHostFailed);
             }
         }
 

Assets/BossRoom/Scripts/Shared/Net/ConnectionManagement/GameNetPortal.cs

@@ -24,14 +24,8 @@ public enum ConnectStatus
         GenericDisconnect,        //server disconnected, but no specific reason given.
         IncompatibleBuildType,    //client build type is incompatible with server.
         HostEndedSession,         //host intentionally ended the session.
-    }
-
-    public enum OnlineMode
-    {
-        IpHost = 0, // The server is hosted directly and clients can join by ip address.
-        UnityRelay = 1, // The server is hosted over a Unity Relay server and clients join by entering a join code.
-        Unset = -1, // The hosting mode is not set yet.
-
+        StartHostFailed,          // server failed to bind
+        StartClientFailed         // failed to connect to server and/or invalid network endpoint
     }
 
     [Serializable]