[DX-Adoption] [RSVP Configs] Better handling of change of cloudType a…

…nd eventType (#365) * Better re-render logic * remove redundant condition
adobecom · Feb 12, 2025 · b44c024 · b44c024
1 parent d5345e6
commit b44c024
Showing 1 changed file with 5 additions and 18 deletions.
diff --git a/ecc/blocks/registration-details-component/controller.js b/ecc/blocks/registration-details-component/controller.js
@@ -222,12 +222,12 @@ function updateHeadingTooltip(component) {
 
 export async function onPayloadUpdate(component, props) {
   const { cloudType, eventType } = props.payload;
-  if (eventType && eventType !== component.dataset.eventType) {
-    component.dataset.eventType = eventType;
-  }
+  const eventTypeChange = eventType && eventType !== component.dataset.eventType;
+  const cloudTypeChange = cloudType && cloudType !== component.dataset.cloudType;
 
-  if (cloudType && cloudType !== component.dataset.cloudType) {
-    component.dataset.cloudType = cloudType;
+  if (eventTypeChange) component.dataset.eventType = eventType;
+  if (eventTypeChange) component.dataset.cloudType = cloudType;
+  if (cloudTypeChange || eventTypeChange) {
     const registrationConfigsWrapper = component.querySelector('.registration-configs-wrapper');
 
     if (!registrationConfigsWrapper) return;
@@ -252,19 +252,6 @@ export async function onPayloadUpdate(component, props) {
     }
 
     updateHeadingTooltip(component);
-  } else if (eventType && eventType !== component.dataset.eventType) {
-    if (cloudType === 'ExperienceCloud') {
-      switch (eventType) {
-        case 'InPerson':
-          buildExperienceCloudInPersonFields(component);
-          break;
-        case 'Webinar':
-          buildExperienceCloudWebinarFields(component);
-          break;
-        default:
-          break;
-      }
-    }
   }
 }