abmp-npm 2.0.3 → 2.0.4

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

@@ -5,11 +5,18 @@ const { changeWixMembersEmails } = require('./utils');
 
 /**
  * Processes and saves multiple member records in bulk
+ * @param {Object} options - The options object
+ * @param {Array} options.memberDataList - Array of member data from API
+ * @param {number} options.currentPageNumber - Current page number being processed
+ * @param {boolean} [options.addInterests=true] - Whether to add interests to the member data
  * @param {Array} memberDataList - Array of member data from API
- * @param {number} currentPageNumber - Current page number being processed
  * @returns {Promise<Object>} - Bulk save operation result with statistics
  */
-const bulkProcessAndSaveMemberData = async (memberDataList, currentPageNumber) => {
+const bulkProcessAndSaveMemberData = async ({
+  memberDataList,
+  currentPageNumber,
+  addInterests = true,
+}) => {
   if (!Array.isArray(memberDataList) || memberDataList.length === 0) {
     throw new Error('Invalid member data list provided');
   }
@@ -18,7 +25,11 @@ const bulkProcessAndSaveMemberData = async (memberDataList, currentPageNumber) =
 
   try {
     const processedMemberDataPromises = memberDataList.map(memberData =>
-      generateUpdatedMemberData(memberData, currentPageNumber)
+      generateUpdatedMemberData({
+        inputMemberData: memberData,
+        currentPageNumber,
+        addInterests,
+      })
     );
 
     const processedMemberDataList = await Promise.all(processedMemberDataPromises);

package/backend/daily-pull/consts.js

@@ -24,8 +24,19 @@ const DEFAULT_MEMBER_DISPLAY_SETTINGS = {
   showWixUrl: true,
 };
 
+const PAC_ASSOCIATIONS = {
+  ABMP: 'ABMP',
+  ASCP: 'ASCP',
+  ANP: 'ANP',
+  AHP: 'AHP',
+};
+
+const SITES_WITH_INTERESTS_AND_SERVICES = [PAC_ASSOCIATIONS.ABMP];
+
 module.exports = {
   MEMBER_ACTIONS,
   ADDRESS_VISIBILITY_OPTIONS,
   DEFAULT_MEMBER_DISPLAY_SETTINGS,
+  PAC_ASSOCIATIONS,
+  SITES_WITH_INTERESTS_AND_SERVICES,
 };

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

@@ -60,11 +60,17 @@ const ensureUniqueUrl = async ({ url, memberId, fullName }) => {
 
 /**
  * Generates complete updated member data by combining existing and migration data
- * @param {Object} inputMemberData - Raw member data from API
- * @param {number} currentPageNumber - Current page number being processed
+ * @param {Object} options - The options object
+ * @param {Object} options.inputMemberData - Raw member data from API
+ * @param {string} options.addInterests - Site association of the member
+ * @param {number} options.currentPageNumber - Current page number being processed
  * @returns {Promise<Object|null>} - Complete updated member data or null if validation fails
  */
-async function generateUpdatedMemberData(inputMemberData, currentPageNumber) {
+async function generateUpdatedMemberData({
+  inputMemberData,
+  addInterests = true,
+  currentPageNumber,
+}) {
   if (!validateCoreMemberData(inputMemberData)) {
     throw new Error(
       'Invalid member data: memberid, email (valid string), and memberships (array) are required'
@@ -86,7 +92,11 @@ async function generateUpdatedMemberData(inputMemberData, currentPageNumber) {
 
   // Only enrich with migration and address data for new members
   if (!existingDbMember) {
-    enrichWithMigrationData(updatedMemberData, inputMemberData.migrationData);
+    enrichWithMigrationData({
+      memberDataToUpdate: updatedMemberData,
+      migrationData: inputMemberData.migrationData,
+      addInterests,
+    });
 
     enrichWithAddressData(
       updatedMemberData,
@@ -196,10 +206,13 @@ async function getNewMemberOnlyFields(inputMemberData, existingDbMember) {
 }
 /**
  * Enriches member data with optional migration properties
- * @param {Object} memberDataToUpdate - Member data object to enhance
+ * @param {Object} options - The options object
+ * @param {Object} options.memberDataToUpdate - Member data object to enhance
+ * @param {Object} options.migrationData - Migration data containing optional properties
+ * @param {boolean} [options.addInterests=true] - Whether to add interests to the member data
  * @param {Object} migrationData - Migration data containing optional properties
  */
-function enrichWithMigrationData(memberDataToUpdate, migrationData) {
+function enrichWithMigrationData({ memberDataToUpdate, migrationData, addInterests = true }) {
   if (!migrationData) return;
 
   memberDataToUpdate.addressInfo = migrationData.addressinfo;
@@ -209,7 +222,7 @@ function enrichWithMigrationData(memberDataToUpdate, migrationData) {
     memberDataToUpdate.showWebsite = true;
   }
 
-  if (migrationData.interests) {
+  if (addInterests && migrationData.interests) {
     memberDataToUpdate.areasOfPractices = processInterests(migrationData.interests);
   }
 }

package/backend/daily-pull/sync-to-cms-methods.js

@@ -6,6 +6,7 @@ const { TASKS_NAMES } = require('../tasks/consts');
 const { getSiteConfigs } = require('../utils');
 
 const { bulkProcessAndSaveMemberData } = require('./bulk-process-methods');
+const { SITES_WITH_INTERESTS_AND_SERVICES } = require('./consts');
 const { isUpdatedMember, isSiteAssociatedMember } = require('./utils');
 
 async function syncMembersDataPerAction(action) {
@@ -79,6 +80,7 @@ async function synchronizeSinglePage(taskObject) {
       getSiteConfigs(CONFIG_KEYS.SITE_ASSOCIATION),
       fetchPACMembers(pageNumber, action),
     ]);
+    const addInterests = SITES_WITH_INTERESTS_AND_SERVICES.includes(siteAssociation);
     if (
       !memberDataResponse ||
       !memberDataResponse.results ||
@@ -98,7 +100,11 @@ async function synchronizeSinglePage(taskObject) {
         message: `No to be updated, or members of association: '${siteAssociation}' found`,
       };
     }
-    const result = await bulkProcessAndSaveMemberData(toSyncMembers, pageNumber);
+    const result = await bulkProcessAndSaveMemberData({
+      memberDataList: toSyncMembers,
+      currentPageNumber: pageNumber,
+      addInterests,
+    });
 
     return {
       success: true,

package/package.json

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