Merge pull request #347 from 0xsequence/Fix/auto-account-federation

Fixed issue where auto-federation was done when email was not already in use

Author: andygruening

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/SequenceLogin.cs

@@ -459,19 +459,20 @@ public async Task ConnectToWaaS(IntentDataOpenSession loginIntent, LoginMethod m
             }
             catch (Exception e)
             {
-                if (e.Message.Contains(EmailInUseError))
+                var emailInUse = e.Message.Contains(EmailInUseError);
+                if (emailInUse)
                 {
                     List<LoginMethod> associatedLoginMethods = ParseLoginMethods(e.Message);
                     OnLoginFailed?.Invoke("Error registering session: " + e.Message, method, email, associatedLoginMethods);
+                    _failedLoginIntent = loginIntent;
+                    _failedLoginMethod = method;
+                    _failedLoginEmail = email;
                 }
                 else
                 {
                     OnLoginFailed?.Invoke("Error registering session: " + e.Message, method, email);
                 }
                 _isLoggingIn = false;
-                _failedLoginIntent = loginIntent;
-                _failedLoginMethod = method;
-                _failedLoginEmail = email;
                 return;
             }
 
@@ -632,6 +633,9 @@ public void ForceCreateAccount()
 
         private async Task ForceCreateWaaSAccount()
         {
+            if (string.IsNullOrEmpty(_failedLoginEmail))
+                throw new Exception("Failed to force create account.");
+            
             _failedLoginIntent.forceCreateAccount = true;
 
             await ConnectToWaaS(_failedLoginIntent, _failedLoginMethod, _failedLoginEmail);

Packages/Sequence-Unity/package.json

@@ -1,6 +1,6 @@
 {
   "name": "xyz.0xsequence.waas-unity",
-  "version": "5.0.10",
+  "version": "5.0.11",
   "displayName": "Sequence SDK",
   "description": "A Unity SDK for Sequence APIs",
   "unity": "2021.3",