reordered reconnect flow to make sure netmanager is shut down before loading main menu

Author: LPLafontaineB

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

@@ -182,17 +182,14 @@ private IEnumerator TryToReconnect()
                 nbTries++;
             }
 
-            if (!NetworkManager.Singleton.IsConnectedClient)
+            // If the coroutine has not been stopped before this, it means we failed to connect during all attempts
+            Debug.Log("All tries failed, returning to main menu");
+            NetworkManager.Singleton.Shutdown();
+            SceneLoaderWrapper.Instance.LoadScene("MainMenu");
+            if (!DisconnectReason.HasTransitionReason)
             {
-                Debug.Log("All tries failed, returning to main menu");
-                SceneLoaderWrapper.Instance.LoadScene("MainMenu");
-                if (!DisconnectReason.HasTransitionReason)
-                {
-                    DisconnectReason.SetDisconnectReason(ConnectStatus.GenericDisconnect);
-                }
-
+                DisconnectReason.SetDisconnectReason(ConnectStatus.GenericDisconnect);
             }
-            NetworkManager.Singleton.Shutdown();
             m_TryToReconnectCoroutine = null;
             m_ConnectStatusPub.Publish(DisconnectReason.Reason);
         }