Added comments pointing to issue numbers in getLoginMethodAndParams

Author: prescottprue

src/utils/auth.js

@@ -77,7 +77,9 @@ export const getLoginMethodAndParams = (firebase, creds) => {
     applicationVerifier,
     credential
   } = creds
+  // Credential Auth
   if (credential) {
+    // Attempt to use signInAndRetrieveDataWithCredential if it exists (see #467 for more info)
     const credentialAuth = firebase.auth().signInAndRetrieveDataWithCredential
 
     if (credentialAuth) {
@@ -88,6 +90,8 @@ export const getLoginMethodAndParams = (firebase, creds) => {
     }
     return { method: 'signInWithCredential', params: [credential] }
   }
+
+  // Provider Auth
   if (provider) {
     // Verify providerName is valid
     if (supportedAuthProviders.indexOf(provider.toLowerCase()) === -1) {
@@ -104,7 +108,10 @@ export const getLoginMethodAndParams = (firebase, creds) => {
     }
     return { method: 'signInWithRedirect', params: [authProvider] }
   }
+
+  // Token Auth
   if (token) {
+    // Check for new sign in method (see #484 for more info)
     const tokenAuth = firebase.auth().signInAndRetrieveDataWithCustomToken
 
     if (tokenAuth) {
@@ -113,6 +120,8 @@ export const getLoginMethodAndParams = (firebase, creds) => {
 
     return { method: 'signInWithCustomToken', params: [token] }
   }
+
+  // Phone Number Auth
   if (phoneNumber) {
     if (!applicationVerifier) {
       throw new Error(
@@ -125,9 +134,10 @@ export const getLoginMethodAndParams = (firebase, creds) => {
     }
   }
 
+  // Email/Password Auth
+  // Check for new sign in method (see #484 for more info)
   const emailPasswordAuth = firebase.auth()
     .signInAndRetrieveDataWithEmailAndPassword
-
   if (emailPasswordAuth) {
     return {
       method: 'signInAndRetrieveDataWithEmailAndPassword',