fix: improve types and apply to ios

invertase · Apr 20, 2024 · 993c696 · 993c696
1 parent bcea81e
commit 993c696
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 10 deletions.
diff --git a/packages/auth/ios/RNFBAuth/RNFBAuthModule.m b/packages/auth/ios/RNFBAuth/RNFBAuthModule.m
@@ -1695,6 +1695,7 @@ - (NSDictionary *)firebaseUserToDict:(FIRUser *)user {
       @"enrollmentTime" : enrollmentTime,
       // @deprecated enrollmentDate kept for backwards compatibility, please use enrollmentTime
       @"enrollmentDate" : enrollmentTime,
+      @"phoneNumber" : hint.phoneNumber,
     }];
   }
   return enrolledFactors;

diff --git a/packages/auth/lib/index.d.ts b/packages/auth/lib/index.d.ts
@@ -472,7 +472,21 @@ export namespace FirebaseAuthTypes {
   /**
    * Contains information about a second factor.
    */
-  export interface MultiFactorInfo {
+  export type MultiFactorInfo = PhoneMultiFactorInfo | TotpMultiFactorInfo;
+
+  export interface PhoneMultiFactorInfo extends MultiFactorInfoCommon {
+    factorId: 'phone';
+    /**
+     * The phone number used for this factor.
+     */
+    phoneNumber: string;
+  }
+
+  export interface TotpMultiFactorInfo extends MultiFactorInfoCommon {
+    factorId: 'totp';
+  }
+
+  export interface MultiFactorInfoCommon {
     /**
      * User friendly name for this factor.
      */
@@ -481,21 +495,12 @@ export namespace FirebaseAuthTypes {
      * Time the second factor was enrolled, in UTC.
      */
     enrollmentTime: string;
-    /**
-     * Type of factor.
-     */
-    factorId: FactorId;
     /**
      * Unique id for this factor.
      */
     uid: string;
   }
 
-  export interface PhoneMultiFactorInfo extends MultiFactorInfo {
-      factorId: "phone";
-      phoneNumber: string;
-  }
-
   export interface MultiFactorAssertion {
     token: string;
     secret: string;