fix import function

SimpleSubs · Aug 24, 2024 · d6c299a · d6c299a
1 parent c79c395
commit d6c299a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/functions/index.js b/functions/index.js
@@ -476,8 +476,8 @@ exports.importUsers = functions.https.onCall(async (data, context) => {
 
   // Write user domain data (only need to use notFound since domain data is already written for found)
   await batchWrite(
-    Object.keys(notFound),
-    (batch, email) => batch.set(userDomainDoc(allUsers[email]), { domain }),
+    notFound.map(email => userDomainDoc(allUsers[email])),
+    (batch, ref) => batch.set(ref, { domain }),
     firestore,
     (error) => errors.push(error)
   );

diff --git a/src/redux/Actions.js b/src/redux/Actions.js
@@ -300,6 +300,8 @@ export function setOrderSchedule(data, dispatch, domain) {
 
 export async function importUsers(data, dispatch, domain) {
   dispatch(setLoading(true));
+
+  // Turns an array of user objects into an object with email as key
   let userData = {};
   for (let user of data) {
     if (user.email) {
@@ -308,6 +310,7 @@ export async function importUsers(data, dispatch, domain) {
       userData[user.email] = thisUser;
     }
   }
+
   let { updated, created, errors } = await importUsersFunction(userData, domain);
   console.log(`Successfully updated ${Object.keys(updated).length} users`);
   console.log(`Successfully created ${Object.keys(created).length} users`);