abmp-npm 1.10.17 → 1.10.18

package/backend/members-data-methods.js

@@ -478,15 +478,25 @@ async function getSiteMemberId(data) {
 }
 
 /**
- * Tracks a button click with member and location info
+ * Tracks a button click with member and location info.
+ * Fetches member data internally using the contact ID.
  * @param {Object} params - Parameters
- * @param {string} params.memberName - Member's full name
- * @param {string} params.memberId - Member's Wix member ID
+ * @param {string} params.contactId - Wix member contact ID (from getCurrentMember()._id)
  * @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
+ * @returns {Promise<Object>} - Saved record or null if member not found
  */
-async function trackButtonClick({ memberName, memberId, pageName, buttonName }) {
+async function trackButtonClick({ contactId, pageName, buttonName }) {
+  const dbMember = await getMemberByContactId(contactId);
+
+  if (!dbMember) {
+    console.warn(`Member not found in MembersDataLatest for contactId: ${contactId}`);
+    return null;
+  }
+
+  const memberName = dbMember.fullName || 'Unknown';
+  const memberId = dbMember.memberId;
+
   const clickData = {
     memberName,
     memberId,

package/package.json

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

package/pages/LearnMore.js

@@ -7,33 +7,21 @@ 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
- * @param {Function} params.getMemberByContactId - Backend function to get member data by contact ID
+ * @param {Function} params.trackClick - Backend function to track the click (handles member lookup internally)
  */
-function learnMoreOnReady({ $w: _$w, trackClick, getMemberByContactId }) {
+function learnMoreOnReady({ $w: _$w, trackClick }) {
   _$w('#learnMoreBtn').onClick(async () => {
     try {
       const wixMember = await members.getCurrentMember();
 
       if (wixMember) {
-        const dbMember = await getMemberByContactId(wixMember._id);
-
-        if (!dbMember) {
-          console.warn('Member not found in MembersDataLatest');
-          return;
-        }
-
-        const memberName = dbMember.fullName || 'Unknown';
-        const memberId = dbMember.memberId;
-
         await trackClick({
-          memberName,
-          memberId,
+          contactId: wixMember._id,
           pageName: PAGE_NAME,
           buttonName: BUTTON_NAME,
         });
 
-        console.log(`Tracked ${BUTTON_NAME} click on ${PAGE_NAME} for member:`, memberId);
+        console.log(`Tracked ${BUTTON_NAME} click on ${PAGE_NAME}`);
       }
     } catch (error) {
       console.error('Error tracking button click:', error);