fix: Click events are not the same, updating event structure

src/payment/checkout/Checkout.jsx

@@ -136,7 +136,6 @@ class Checkout extends React.Component {
       title: PaymentTitle,
       url: window.location.href,
       pageType: 'checkout',
-      elementType: ElementType.Button,
       timestamp: Date.now(),
       productList: this.getProductList(),
     };
@@ -170,7 +169,7 @@ class Checkout extends React.Component {
     if (products || products.length !== 0) {
       products.forEach(product => {
         productList.push({
-          variant: BaseTagularVariant.Courses,
+          variant: BaseTagularVariant.Courses, // TODO: all are 'courses' now, use translateVariant() in the future
           brand: this.getPartnerName(product), // School or Partner name
           name: product.title, // Course(s) title
         });

src/payment/data/actions.js

@@ -80,18 +80,30 @@ export const TRACK_PAYMENT_BUTTON_CLICK = 'TRACK_PAYMENT_BUTTON_CLICK';
 
 export const trackPaymentButtonClick = tagularElement => {
   // Ideally this would happen in a middleware saga for separation of concerns
-  // but due to deadlines/payment MFE will go away, adding a call here
-  const conversionEvent = {
-    correlation: {
-      id: getCorrelationID(),
-    },
-    metadata: tagularElement,
-  };
-  tagularEvent('ConversionTracked', conversionEvent);
+  // but due to deadlines/payment MFE will go away, adding a call here.
+  // Note: Click events on the PayPal button and Place Order button differ in the type of event data it's treated as.
+  let payload;
+  if (tagularElement.name === 'paypal') {
+    payload = {
+      correlation: {
+        id: getCorrelationID(),
+      },
+      webElement: tagularElement,
+    };
+    tagularEvent('ElementClicked', payload);
+  } else {
+    payload = {
+      correlation: {
+        id: getCorrelationID(),
+      },
+      metadata: tagularElement,
+    };
+    tagularEvent('ConversionTracked', payload);
+  }
 
   return {
     type: TRACK_PAYMENT_BUTTON_CLICK,
-    payload: conversionEvent,
+    payload,
   };
 };