abmp-npm 1.1.90 → 1.1.91

package/backend/contacts-methods.js

@@ -3,7 +3,33 @@ const { auth } = require('@wix/essentials');
 
 const elevatedGetContact = auth.elevate(contacts.getContact);
 const elevatedUpdateContact = auth.elevate(contacts.updateContact);
+const elevatedCreateContact = auth.elevate(contacts.createContact);
 
+/**
+ * Create a contact in Wix CRM
+ * @param {Object} contactData - Contact data
+ * @param {boolean} allowDuplicates - Allow duplicates if contact with same email already exists, will be true only when handling existing members, after that should be removed
+ * @returns {Promise<Object>} - Contact data
+ */
+async function createContact(contactData, allowDuplicates = false) {
+  if (!contactData || !(contactData.contactFormEmail || contactData.email)) {
+    throw new Error('Contact data is required');
+  }
+  const phones =
+    Array.isArray(contactData.phones) && contactData.phones.length > 0 ? contactData.phones : [];
+  const contactInfo = {
+    name: {
+      first: contactData.firstName,
+      last: contactData.lastName,
+    },
+    emails: [
+      { items: [{ email: contactData.contactFormEmail || contactData.email, primary: true }] },
+    ],
+    phones: [{ items: phones.map(phone => ({ phone })) }],
+  };
+  const createContactResponse = await elevatedCreateContact(contactInfo, { allowDuplicates });
+  return createContactResponse.contact._id;
+}
 /**
  * Generic contact update helper function
  * @param {string} contactId - The contact ID in Wix CRM
@@ -14,18 +40,13 @@ async function updateContactInfo(contactId, updateInfoCallback, operationName) {
   if (!contactId) {
     throw new Error('Contact ID is required');
   }
-  console.log('updateContactInfo contactId', contactId);
-  console.log('updateContactInfo operationName', operationName);
+
   try {
     const contact = await elevatedGetContact(contactId);
-    console.log('updateContactInfo contact', contact);
     const currentInfo = contact.info;
-    console.log('updateContactInfo currentInfo', currentInfo);
     const updatedInfo = updateInfoCallback(currentInfo);
-    console.log('updateContactInfo updatedInfo', updatedInfo);
-    const updatedContact = await elevatedUpdateContact(contactId, updatedInfo, contact.revision);
-    console.log('updateContactInfo updatedContact', updatedContact);
-    return updatedContact;
+
+    await elevatedUpdateContact(contactId, updatedInfo, contact.revision);
   } catch (error) {
     console.error(`Error in ${operationName}:`, error);
     throw new Error(`Failed to ${operationName}: ${error.message}`);
@@ -38,8 +59,6 @@ async function updateContactInfo(contactId, updateInfoCallback, operationName) {
  * @param {string} newEmail - The new email address
  */
 async function updateContactEmail(contactId, newEmail) {
-  console.log('updateContactEmail contactId', contactId);
-  console.log('updateContactEmail newEmail', newEmail);
   if (!newEmail) {
     throw new Error('New email is required');
   }
@@ -62,27 +81,33 @@ async function updateContactEmail(contactId, newEmail) {
 }
 
 /**
- * Updates contact names in Wix CRM
- * @param {string} contactId - The contact ID in Wix CRM
- * @param {string} firstName - The new first name
- * @param {string} lastName - The new last name
+ * Updates contact names in Wix CRM for both contact and member
+ * @param {Object} params - Parameters object
+ * @param {string} params.wixContactId - The contact ID in Wix CRM
+ * @param {string} params.wixMemberId - The member ID in Wix CRM
+ * @param {string} params.firstName - The new first name
+ * @param {string} params.lastName - The new last name
  */
-async function updateContactNames(contactId, firstName, lastName) {
+async function updateMemberAndContactNames({ wixContactId, wixMemberId, firstName, lastName }) {
+  //TODO: rethink if we should keep all info just in contact, meaning no need to update the member
   if (!firstName && !lastName) {
     throw new Error('At least one name field is required');
   }
 
-  return await updateContactInfo(
-    contactId,
-    currentInfo => ({
-      ...currentInfo,
-      name: {
-        first: firstName || currentInfo?.name?.first || '',
-        last: lastName || currentInfo?.name?.last || '',
-      },
-    }),
-    'update contact names'
-  );
+  const createNameUpdate = currentInfo => ({
+    ...currentInfo,
+    name: {
+      first: firstName || currentInfo?.name?.first || '',
+      last: lastName || currentInfo?.name?.last || '',
+    },
+  });
+
+  const updatePromises = [
+    wixContactId && updateContactInfo(wixContactId, createNameUpdate, 'update contact names'),
+    wixMemberId && updateContactInfo(wixMemberId, createNameUpdate, 'update member names'),
+  ].filter(Boolean);
+
+  return await Promise.all(updatePromises);
 }
 
 /**
@@ -95,7 +120,6 @@ async function updateContactNames(contactId, firstName, lastName) {
 const updateIfChanged = (existingValues, newValues, updater, argsBuilder) => {
   const hasChanged = existingValues.some((val, idx) => val !== newValues[idx]);
   if (!hasChanged) return null;
-  console.log('updateIfChanged hasChanged', hasChanged);
   return updater(...argsBuilder(newValues));
 };
 
@@ -105,18 +129,18 @@ const updateIfChanged = (existingValues, newValues, updater, argsBuilder) => {
  * @param {Object} existingMemberData - Existing member data
  */
 const updateMemberContactInfo = async (data, existingMemberData) => {
-  const { contactId } = existingMemberData;
-  console.log('updateMemberContactInfo contactId', contactId);
+  const { wixContactId, wixMemberId } = existingMemberData;
+
   const updateConfig = [
     {
       fields: ['contactFormEmail'],
       updater: updateContactEmail,
-      args: ([email]) => [contactId, email],
+      args: ([email]) => [wixContactId, email],
     },
     {
       fields: ['firstName', 'lastName'],
-      updater: updateContactNames,
-      args: ([firstName, lastName]) => [contactId, firstName, lastName],
+      updater: updateMemberAndContactNames,
+      args: ([firstName, lastName]) => [{ firstName, lastName, wixContactId, wixMemberId }],
     },
   ];
 
@@ -128,11 +152,10 @@ const updateMemberContactInfo = async (data, existingMemberData) => {
     })
     .filter(Boolean);
 
-  const resp = await Promise.all(updatePromises);
-  console.log('updateMemberContactInfo updatePromises', resp);
-  return resp;
+  await Promise.all(updatePromises);
 };
 
 module.exports = {
   updateMemberContactInfo,
+  createContact,
 };

package/backend/daily-pull/bulk-process-methods.js

@@ -168,7 +168,7 @@ const bulkProcessAndSaveMemberData = async ({
     const toChangeWixMembersEmails = [];
     const toSaveMembersData = uniqueUrlsMembersData.map(member => {
       const { isLoginEmailChanged, isNewToDb: _isNewToDb, ...restMemberData } = member;
-      if (member.contactId && isLoginEmailChanged) {
+      if (member.wixMemberId && isLoginEmailChanged) {
         toChangeWixMembersEmails.push(member);
       }
       return restMemberData; //we don't want to store the isLoginEmailChanged in the database, it's just a flag to know if we need to change the login email in Members area

package/backend/daily-pull/utils.js

@@ -35,7 +35,17 @@ const extractBaseUrl = url => {
   return url;
 };
 const incrementUrlCounter = (existingUrl, baseUrl) => {
-  if (existingUrl && existingUrl === baseUrl) {
+  if (!existingUrl || !baseUrl) {
+    return baseUrl;
+  }
+  // Normalize for comparison (case-insensitive)
+  const normalizedExisting = existingUrl.toLowerCase();
+  const normalizedBase = baseUrl.toLowerCase();
+
+  if (
+    normalizedExisting === normalizedBase ||
+    normalizedExisting.startsWith(`${normalizedBase}-`)
+  ) {
     console.log(
       `Found member with same url ${existingUrl} for baseUrl ${baseUrl}, increasing counter by 1`
     );
@@ -44,6 +54,9 @@ const incrementUrlCounter = (existingUrl, baseUrl) => {
     const lastCounter = isNumeric ? parseInt(lastSegment, 10) : 0;
     return `${baseUrl}-${lastCounter + 1}`;
   }
+
+  // No conflict, return baseUrl with counter 1 to be safe
+  return `${baseUrl}-1`;
 };
 /**
  * Validates core member data requirements

package/backend/dev-only-methods.js

@@ -1,3 +1,5 @@
+const { COLLECTIONS } = require('../public/consts');
+
 const { ensureUniqueUrlsInBatch } = require('./daily-pull/bulk-process-methods');
 const { wixData } = require('./elevated-modules');
 const { bulkSaveMembers } = require('./members-data-methods');
@@ -15,4 +17,14 @@ async function deduplicateURls(collectionName, duplicateUrlsList) {
   return await bulkSaveMembers(membersWithUniqueUrls, collectionName);
 }
 
-module.exports = { deduplicateURls };
+async function copyContactIdToWixMemberId() {
+  const query = wixData.query(COLLECTIONS.MEMBERS_DATA).isNotEmpty('contactId');
+  const members = await queryAllItems(query);
+  const updatedMembers = members.map(member => ({
+    ...member,
+    wixMemberId: member.contactId,
+  }));
+  return await bulkSaveMembers(updatedMembers, COLLECTIONS.MEMBERS_DATA);
+}
+
+module.exports = { deduplicateURls, copyContactIdToWixMemberId };

package/backend/forms-methods.js

@@ -16,7 +16,7 @@ const contactSubmission = async (data, memberDataId) => {
     console.log('Member contact form is not enabled for user, skipping contact submission!');
     return;
   }
-  let memberContactId = memberData.contactId;
+  let memberContactId = memberData.wixContactId;
   if (!memberContactId) {
     /**
      * Create a member contact here since some members may have never logged in
@@ -27,7 +27,7 @@ const contactSubmission = async (data, memberDataId) => {
      */
     console.info('Member contact id not found for user, creating new contact!');
     const member = await createContactAndMemberIfNew(memberData);
-    memberContactId = member.contactId;
+    memberContactId = member.wixContactId;
   }
   console.log('memberContactId', memberContactId);
   const emailTriggered = await triggerAutomation(automationEmailTriggerId, {
@@ -40,7 +40,7 @@ const contactSubmission = async (data, memberDataId) => {
   data = {
     ...data,
     phone: Number(data.phone),
-    memberContactId: memberContactId,
+    memberContactId,
     memberEmail: memberData.contactFormEmail,
   };
   await wixData.insert(COLLECTIONS.CONTACT_US_SUBMISSIONS, data);

package/backend/members-area-methods.js

@@ -2,10 +2,11 @@ const { auth } = require('@wix/essentials');
 const { members, authentication } = require('@wix/members');
 const elevatedCreateMember = auth.elevate(members.createMember);
 
-function prepareContactData(partner) {
+function prepareMemberData(partner) {
   const phones = Array.isArray(partner.phones) ? partner.phones : []; //some users don't have phones
   const options = {
     member: {
+      //Keeping contact creation in member data for future purposes, in case we need to use it later
       contact: {
         ...partner,
         phones,
@@ -22,9 +23,8 @@ async function createMemberFunction(member) {
 }
 const createSiteMember = async memberDetails => {
   try {
-    const options = prepareContactData(memberDetails);
-    const contactId = await createMemberFunction(options);
-    return contactId;
+    const options = prepareMemberData(memberDetails);
+    return await createMemberFunction(options);
   } catch (error) {
     console.error(`Error in createSiteMember ${error.message}`);
     throw error;
@@ -37,22 +37,25 @@ const getCurrentMember = async () => {
 };
 
 /**
- * Updates Wix member login email if the member has a contactId (registered Wix member)
- * @param {Object} member - Member object with contactId and email
+ * Updates Wix member login email if the member has a wixMemberId (registered Wix member)
+ * @param {Object} member - Member object with wixMemberId and email
  * @param {Object} result - Result object to track Wix member updates
  */
 async function updateWixMemberLoginEmail(member, result = {}) {
-  if (!member.contactId) {
-    console.log(`Member ${member.memberId} has no contactId - skipping Wix login email update`);
+  if (!member.wixMemberId) {
+    console.log(`Member ${member.memberId} has no wixMemberId - skipping Wix login email update`);
     return;
   }
 
   try {
     console.log(
-      `Updating Wix login email for member ${member.memberId} (contactId: ${member.contactId})`
+      `Updating Wix login email for member ${member.memberId} (wixMemberId: ${member.wixMemberId})`
     );
 
-    const updatedWixMember = await authentication.changeLoginEmail(member.contactId, member.email);
+    const updatedWixMember = await authentication.changeLoginEmail(
+      member.wixMemberId,
+      member.email
+    );
 
     console.log(
       `✅ Successfully updated Wix login email for member ${member.memberId}: ${updatedWixMember.loginEmail}`
@@ -75,7 +78,7 @@ async function updateWixMemberLoginEmail(member, result = {}) {
     }
     result.wixMemberErrors.push({
       memberId: member.memberId,
-      contactId: member.contactId,
+      wixMemberId: member.wixMemberId,
       email: member.email,
       error: error.message,
     });

package/backend/members-data-methods.js

@@ -1,10 +1,10 @@
 const { COLLECTIONS } = require('../public/consts');
 
 const { MEMBERSHIPS_TYPES } = require('./consts');
-const { updateMemberContactInfo } = require('./contacts-methods');
+const { updateMemberContactInfo, createContact } = require('./contacts-methods');
 const { MEMBER_ACTIONS } = require('./daily-pull/consts');
 const { wixData } = require('./elevated-modules');
-const { createSiteMember } = require('./members-area-methods');
+const { createSiteMember, getCurrentMember } = require('./members-area-methods');
 const {
   chunkArray,
   normalizeUrlForComparison,
@@ -42,10 +42,14 @@ async function createContactAndMemberIfNew(memberData) {
       phones: memberData.phones,
       contactFormEmail: memberData.contactFormEmail || memberData.email,
     };
-    const contactId = await createSiteMember(toCreateMemberData);
+    const [wixMemberId, wixContactId] = await Promise.all([
+      createSiteMember(toCreateMemberData),
+      createContact(toCreateMemberData),
+    ]);
     let memberDataWithContactId = {
       ...memberData,
-      contactId,
+      wixMemberId,
+      wixContactId,
     };
     const updatedResult = await updateMember(memberDataWithContactId);
     memberDataWithContactId = {
@@ -498,9 +502,60 @@ async function prepareMemberForQALogin(email) {
   }
 }
 
+/**
+ * Tracks a button click with member and location info.
+ * @param {Object} params - Parameters
+ * @param {string} params.pageName - Name of the page/popup where button was clicked
+ * @param {string} params.buttonName - Name/ID of the button that was clicked
+ * @returns {Promise<Object>} - Saved record or null if member not found
+ */
+async function trackButtonClick({ pageName, buttonName }) {
+  const wixMember = await getCurrentMember();
+
+  if (!wixMember) {
+    console.warn('[trackButtonClick]: No logged in member found');
+    return null;
+  }
+
+  const dbMember = await getMemberByContactId(wixMember._id);
+
+  if (!dbMember) {
+    console.warn(
+      `[trackButtonClick]: Member not found in MembersDataLatest for contactId: ${wixMember._id}`
+    );
+    return null;
+  }
+
+  const memberName = dbMember.fullName || 'Unknown';
+  const memberId = dbMember.memberId;
+
+  const clickData = {
+    memberName,
+    memberId,
+    pageName,
+    buttonName,
+    clickedAt: new Date(),
+  };
+
+  try {
+    const result = await wixData.insert(COLLECTIONS.BUTTON_CLICKS, clickData);
+    console.log(`Tracked ${buttonName} click on ${pageName} for member ${memberId}`);
+    return result;
+  } catch (error) {
+    console.error(`Error tracking ${buttonName} click:`, error);
+    throw error;
+  }
+}
+
+async function getAllMembersWithWixMemberId() {
+  const membersQuery = wixData.query(COLLECTIONS.MEMBERS_DATA).isNotEmpty('wixMemberId');
+  return await queryAllItems(membersQuery);
+}
+
 module.exports = {
   findMemberByWixDataId,
   createContactAndMemberIfNew,
+  getAllMembersWithWixMemberId,
   saveRegistrationData,
   bulkSaveMembers,
   findMemberById,
@@ -518,4 +573,5 @@ module.exports = {
   prepareMemberForSSOLogin,
   prepareMemberForQALogin,
   checkUrlUniqueness,
+  trackButtonClick,
 };

package/backend/tasks/consts.js

@@ -16,6 +16,8 @@ const TASKS_NAMES = {
   migrateUrlsChunk: 'migrateUrlsChunk',
   scheduleGenerateMissingUrls: 'scheduleGenerateMissingUrls',
   generateUrlsChunk: 'generateUrlsChunk',
+  scheduleCreateContactsFromMembers: 'scheduleCreateContactsFromMembers',
+  createContactsFromMembers: 'createContactsFromMembers',
 };
 
 module.exports = {

package/backend/tasks/tasks-configs.js

@@ -16,6 +16,8 @@ const {
   migrateContactFormEmails,
   scheduleEmailSync,
   syncMemberLoginEmails,
+  scheduleCreateContactsFromMembers,
+  createContactsFromMembers,
 } = require('./tasks-process-methods');
 const {
   scheduleMigrateExistingUrls,
@@ -153,6 +155,20 @@ const TASKS = {
     shouldSkipCheck: () => false,
     estimatedDurationSec: 80,
   },
+  [TASKS_NAMES.scheduleCreateContactsFromMembers]: {
+    name: TASKS_NAMES.scheduleCreateContactsFromMembers,
+    getIdentifier: () => 'SHOULD_NEVER_SKIP',
+    process: scheduleCreateContactsFromMembers,
+    shouldSkipCheck: () => false,
+    estimatedDurationSec: 80,
+  },
+  [TASKS_NAMES.createContactsFromMembers]: {
+    name: TASKS_NAMES.createContactsFromMembers,
+    getIdentifier: task => task.data,
+    process: createContactsFromMembers,
+    shouldSkipCheck: () => false,
+    estimatedDurationSec: 80,
+  },
 };
 
 module.exports = { TASKS };

package/backend/tasks/tasks-process-methods.js

@@ -13,6 +13,8 @@ const {
   bulkSaveMembers,
   getAllUpdatedLoginEmails,
   getMembersByIds,
+  createContactAndMemberIfNew,
+  getAllMembersWithWixMemberId,
 } = require('../members-data-methods');
 const {
   getCompleteStateList,
@@ -530,6 +532,46 @@ const syncMemberLoginEmails = async data => {
     throw new Error(errorMessage);
   }
 };
+/**
+ * Schedules tasks to create contacts from members
+ * dev-only task, run only once by the developers
+ */
+const scheduleCreateContactsFromMembers = async () => {
+  const members = await getAllMembersWithWixMemberId();
+  console.log(
+    `Starting to schedule create contacts from members tasks for ${members.length} members in chunks of 500 members`
+  );
+  const membersChunks = chunkArray(members, 500);
+  for (let chunkIndex = 0; chunkIndex < membersChunks.length; chunkIndex++) {
+    const chunk = membersChunks[chunkIndex];
+    const toScheduleTask = {
+      name: TASKS_NAMES.createContactsFromMembers,
+      data: { chunk, chunkIndex },
+    };
+    await taskManager().schedule(toScheduleTask);
+    console.log(`Scheduled task for chunk ${chunkIndex} with ${chunk.length} members`);
+  }
+  console.log(`Successfully scheduled ${membersChunks.length} tasks for ${members.length} members`);
+};
+
+/**
+ * Creates contacts from members
+ * dev-only task, run only once by the developers
+ */
+const createContactsFromMembers = async data => {
+  const { chunk, chunkIndex } = data;
+  console.log(`Creating contacts from ${chunk.length} members in chunk ${chunkIndex}`);
+  const createPromises = chunk.map(member => createContactAndMemberIfNew(member));
+  const createResults = await Promise.all(createPromises);
+  console.log(
+    `Created ${createResults.length} contacts from ${chunk.length} members in chunk ${chunkIndex}`
+  );
+  const saveResult = await bulkSaveMembers(createResults);
+  console.log(
+    `Successfully saved ${saveResult.totalSaved} contacts from ${chunk.length} members in chunk ${chunkIndex}`
+  );
+  return saveResult;
+};
 
 module.exports = {
   scheduleTaskForEmptyAboutYouMembers,
@@ -542,4 +584,6 @@ module.exports = {
   migrateContactFormEmails,
   scheduleEmailSync,
   syncMemberLoginEmails,
+  scheduleCreateContactsFromMembers, // run only once by the developers
+  createContactsFromMembers,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "abmp-npm",
-  "version": "1.1.90",
+  "version": "1.1.91",
   "main": "index.js",
   "scripts": {
     "check-cycles": "madge --circular .",

package/pages/Home.js

@@ -216,12 +216,19 @@ const homePageOnReady = async ({
         $item('#milesAwayText').text = '';
       }
 
-      // 7) "Show maps" button enabled only if there's at least one visible address
+      // 7) "Show maps" button enabled only if there's a full address with valid coordinates
       const visible = checkAddressIsVisible(addresses);
-      if (visible.length && visible[0].addressStatus === ADDRESS_STATUS_TYPES.FULL_ADDRESS) {
+      const fullAddressWithValidCoords = visible.find(
+        addr =>
+          addr.addressStatus === ADDRESS_STATUS_TYPES.FULL_ADDRESS &&
+          addr.latitude &&
+          addr.longitude
+      );
+
+      if (fullAddressWithValidCoords) {
         $item('#showMaps').enable();
         $item('#showMaps').show();
-        const { latitude, longitude } = visible[0];
+        const { latitude, longitude } = fullAddressWithValidCoords;
         $item('#showMaps').link = `https://maps.google.com/?q=${latitude},${longitude}`;
         $item('#showMaps').target = '_blank';
       } else {

package/pages/LearnMore.js

@@ -0,0 +1,27 @@
+const PAGE_NAME = 'Learn More';
+const BUTTON_NAME = 'Upgrade Now';
+
+/**
+ * Creates the Learn More popup handler
+ * @param {Object} params - Parameters
+ * @param {Function} params.$w - Wix $w selector
+ * @param {Function} params.trackClick - Backend function to track the click (handles member lookup internally)
+ */
+function learnMoreOnReady({ $w: _$w, trackClick }) {
+  _$w('#learnMoreBtn').onClick(async () => {
+    try {
+      await trackClick({
+        pageName: PAGE_NAME,
+        buttonName: BUTTON_NAME,
+      });
+
+      console.log(`Tracked ${BUTTON_NAME} click on ${PAGE_NAME}`);
+    } catch (error) {
+      console.error('Error tracking button click:', error);
+    }
+  });
+}
+
+module.exports = {
+  learnMoreOnReady,
+};

package/pages/index.js

@@ -8,4 +8,5 @@ module.exports = {
   ...require('./SelectBannerImages.js'),
   ...require('./deleteConfirm.js'),
   ...require('./SaveAlerts.js'),
+  ...require('./LearnMore.js'),
 };

package/public/consts.js

@@ -13,6 +13,7 @@ const COLLECTIONS = {
   STATE_CITY_MAP: 'City',
   UPDATED_LOGIN_EMAILS: 'updatedLoginEmails',
   QA_USERS: 'QA_Users', //Make QA users configurable per site
+  BUTTON_CLICKS: 'ButtonClicks',
 };
 
 /**