abmp-npm 2.0.14 → 2.0.15

package/backend/cms-data-methods.js

@@ -8,6 +8,7 @@ const {
   GEO_HASH_PRECISION,
   MAX__MEMBERS_SEARCH_RESULTS,
   WIX_QUERY_MAX_LIMIT,
+  MEMBERSHIPS_TYPES,
 } = require('./consts.js');
 const { wixData } = require('./elevated-modules');
 
@@ -31,6 +32,7 @@ function buildMembersSearchQuery(data) {
         .query(COLLECTIONS.MEMBERS_DATA)
         .ne('optOut', true)
         .ne('action', 'drop')
+        .ne('memberships.membertype', MEMBERSHIPS_TYPES.PAC_STAFF)
         .eq('isVisible', true);
       let filterConfig = [
         {
@@ -105,7 +107,7 @@ function buildMembersSearchQuery(data) {
         query = query.contains('fullName', filter.searchText);
       }
       if (!includeStudents) {
-        query = query.ne('memberships.membertype', 'Student');
+        query = query.ne('memberships.membertype', MEMBERSHIPS_TYPES.STUDENT);
       }
       return query;
     },

package/backend/tasks/url-migration-methods.js

@@ -9,7 +9,8 @@ const { queryAllItems, chunkArray } = require('../utils');
 const { TASKS_NAMES } = require('./consts');
 
 const COLLECTION_WITH_URLS = 'MembersDataWithUrls';
-const CHUNK_SIZE = 5000; // 5k members per task
+const CHUNK_SIZE = 5000; // 5k members per task for migration
+const GENERATION_CHUNK_SIZE = 1000; // 1k members per task for URL generation
 
 /**
  * Step 1: Migrate existing URLs from backup collection
@@ -201,7 +202,7 @@ async function scheduleGenerateMissingUrls() {
       };
     }
 
-    const chunks = chunkArray(membersToUpdate, CHUNK_SIZE);
+    const chunks = chunkArray(membersToUpdate, GENERATION_CHUNK_SIZE);
 
     for (let i = 0; i < chunks.length; i++) {
       const chunk = chunks[i];
@@ -255,11 +256,25 @@ async function generateUrlsChunk(data) {
   };
 
   try {
-    // Fetch all members at once using hasSome
-    console.log(`Fetching ${memberIds.length} members from database...`);
-    const members = await queryAllItems(
-      wixData.query(COLLECTIONS.MEMBERS_DATA).hasSome('_id', memberIds)
+    // Fetch members in smaller batches to avoid cursor size limits
+    // hasSome with too many IDs creates cursors that exceed Wix's 150KB limit
+    const FETCH_BATCH_SIZE = 200;
+    console.log(
+      `Fetching ${memberIds.length} members from database in batches of ${FETCH_BATCH_SIZE}...`
     );
+
+    const members = [];
+    const idBatches = chunkArray(memberIds, FETCH_BATCH_SIZE);
+
+    for (let i = 0; i < idBatches.length; i++) {
+      const idBatch = idBatches[i];
+      const batchMembers = await queryAllItems(
+        wixData.query(COLLECTIONS.MEMBERS_DATA).hasSome('_id', idBatch)
+      );
+      members.push(...batchMembers);
+      console.log(`Fetched batch ${i + 1}/${idBatches.length}: ${batchMembers.length} members`);
+    }
+
     console.log(`Found ${members.length} members in database`);
 
     // Create a map of _id -> member for quick lookup
@@ -292,7 +307,7 @@ async function generateUrlsChunk(data) {
       try {
         const uniqueUrl = await ensureUniqueUrl({
           url: '',
-          memberId: member._id,
+          memberId: member.memberId,
           fullName: name || '', // Let ensureUniqueUrl handle fallback for empty names
         });
 

package/package.json

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

package/pages/personalDetails.js

@@ -1294,6 +1294,7 @@ async function personalDetailsOnReady({
       streetAddress: {
         name: extractStreetName(address.line1),
         number: extractStreetNumber(address.line1),
+        apt: address.line2 || '',
       },
       city: address.city || '',
       subdivision: address.state || '',
@@ -1309,20 +1310,49 @@ async function personalDetailsOnReady({
     if (!addressInputValue) return null;
 
     let line1 = '';
+    let line2 = '';
+
     if (addressInputValue.streetAddress) {
       const number = addressInputValue.streetAddress.number || '';
       const name = addressInputValue.streetAddress.name || '';
       line1 = `${number} ${name}`.trim();
+
+      // Capture apartment/suite/building info from streetAddress.apt (undocumented but exists)
+      if (addressInputValue.streetAddress.apt) {
+        line2 = addressInputValue.streetAddress.apt;
+      }
     }
 
     if (!line1 && addressInputValue.formatted) {
       line1 = addressInputValue.formatted.split(',')[0]?.trim() || '';
     }
 
+    // If line2 is still empty, try to extract building/suite info from formatted address
+    if (!line2 && addressInputValue.formatted) {
+      const formattedParts = addressInputValue.formatted.split(',').map(part => part.trim());
+      // Look for BLDG/STE/APT/UNIT/SUITE info in the formatted parts
+      for (let i = 1; i < formattedParts.length; i++) {
+        const part = formattedParts[i];
+        const lowerPart = part.toLowerCase();
+        if (
+          lowerPart.includes('bldg') ||
+          lowerPart.includes('ste') ||
+          lowerPart.includes('apt') ||
+          lowerPart.includes('unit') ||
+          lowerPart.includes('suite') ||
+          lowerPart.includes('#') ||
+          lowerPart.includes('building')
+        ) {
+          line2 = part;
+          break;
+        }
+      }
+    }
+
     return {
       key: existingAddress?.key || generateId(),
       line1,
-      line2: existingAddress?.line2 || '',
+      line2: line2 || existingAddress?.line2 || '',
       city: addressInputValue.city || '',
       state: addressInputValue.subdivision || '',
       postalcode: addressInputValue.postalCode || '',
@@ -1705,6 +1735,7 @@ async function personalDetailsOnReady({
     const parts = [];
 
     if (addr.line1) parts.push(addr.line1);
+    if (addr.line2) parts.push(addr.line2); // Include building/suite info
     if (addr.city) parts.push(addr.city);
     if (addr.state && addr.postalcode) {
       parts.push(`${addr.state} ${addr.postalcode}`);

package/public/Utils/sharedUtils.js

@@ -91,7 +91,7 @@ function formatAddress(item) {
   const limitedPostalCode = item.postalcode.slice(0, 5); //show only 5 digits to not show full user address
   switch (item.addressStatus) {
     case ADDRESS_STATUS_TYPES.FULL_ADDRESS:
-      addressParts = [item.line1, item.city, item.state, limitedPostalCode];
+      addressParts = [item.line1, item.line2, item.city, item.state, limitedPostalCode];
       break;
     case ADDRESS_STATUS_TYPES.STATE_CITY_ZIP:
       addressParts = [item.city, item.state, limitedPostalCode];