@capillarytech/creatives-library 9.0.56-alpha.7 → 9.0.56-betaa.0

package/v2Containers/CommunicationFlow/Tests/CommunicationFlowCard.test.js

@@ -108,22 +108,6 @@ describe('CommunicationFlowCard', () => {
       fireEvent.click(screen.getByText('Add creatives'));
       expect(screen.getByTestId('comm-flow-mock')).toHaveAttribute('data-mode', 'create');
     });
-
-    it('disables the Add creatives button when disabled prop is true', () => {
-      renderCard({ disabled: true, disabledTooltip: 'Enter an Alert name to add content' });
-      expect(screen.getByText('Add creatives').closest('button')).toBeDisabled();
-    });
-
-    it('does not open SlideBox when Add creatives is clicked while disabled', () => {
-      renderCard({ disabled: true, disabledTooltip: 'Enter an Alert name to add content' });
-      fireEvent.click(screen.getByText('Add creatives'));
-      expect(screen.queryByTestId('slide-box')).not.toBeInTheDocument();
-    });
-
-    it('Add creatives button is enabled by default (disabled prop absent)', () => {
-      renderCard();
-      expect(screen.getByText('Add creatives').closest('button')).not.toBeDisabled();
-    });
   });
 
   describe('configured state (initialData provided)', () => {
@@ -202,26 +186,6 @@ describe('CommunicationFlowCard', () => {
       renderCard({ initialData: makeSavedData() });
       expect(screen.getByText('Other controls')).toBeInTheDocument();
     });
-
-    it('hides the Other controls column when config.features.showDynamicControls is false', () => {
-      renderCard({
-        config: { ...baseConfig, features: { showDynamicControls: false } },
-        initialData: makeSavedData({
-          dynamicControls: {
-            sendToControlCustomers: false, sendToBrandPocs: false, useTinyUrl: false, overrideDailyLimit: false,
-          },
-        }),
-      });
-      expect(screen.queryByText('Other controls')).not.toBeInTheDocument();
-    });
-
-    it('still shows the Other controls column when showDynamicControls is unset (Engagement Module default)', () => {
-      renderCard({
-        config: { ...baseConfig, features: {} },
-        initialData: makeSavedData(),
-      });
-      expect(screen.getByText('Other controls')).toBeInTheDocument();
-    });
   });
 
   describe('channel config resolution', () => {
@@ -267,11 +231,11 @@ describe('CommunicationFlowCard', () => {
       expect(screen.getByText(/Sender ID/)).toBeInTheDocument();
     });
 
-    it('shows senderId for EMAIL with Sender ID label', () => {
+    it('shows senderEmail for EMAIL with Sender ID label', () => {
       renderCard({
         initialData: makeSavedData({
           contentItems: [{ channel: 'EMAIL', templateData: {} }],
-          deliverySetting: { channelSetting: { EMAIL: { senderId: 'test@example.com' } } },
+          deliverySetting: { channelSetting: { EMAIL: { senderEmail: 'test@example.com' } } },
         }),
       });
       expect(screen.getByText(/test@example.com/)).toBeInTheDocument();
@@ -581,10 +545,10 @@ describe('CommunicationFlowCard', () => {
   });
 
   describe('SlideBox header', () => {
-    it('renders Add content title in the slidebox header', () => {
+    it('renders Add message title in the slidebox header', () => {
       renderCard();
       fireEvent.click(screen.getByText('Add creatives'));
-      expect(screen.getByText('Add content')).toBeInTheDocument();
+      expect(screen.getByText('Add message')).toBeInTheDocument();
     });
   });
 

package/v2Containers/CommunicationFlow/constants.js

@@ -38,68 +38,9 @@ export const CHANNEL_PRIORITY = 'CHANNEL_PRIORITY';
 export const AB_TEST = 'AB_TEST';
 export const SINGLE_TEMPLATE = 'SINGLE_TEMPLATE';
 
-// CCS create is only supported for SINGLE (CHANNEL_PRIORITY/AB_TEST skip CCS entirely).
-export const CCS_STRATEGY_TYPE_SINGLE = 'SINGLE';
-
-// Mirrors CCS SingleChannelStrategy field names; kept separate from the legacy
-// messageMeta CHANNEL_CONTENT_KEY_MAP/CHANNEL_DELIVERY_KEY_MAP.
-export const CCS_CHANNEL_CONTENT_KEY_MAP = {
-  SMS: 'smsMessageContent',
-  EMAIL: 'emailMessageContent',
-  WHATSAPP: 'whatsappMessageContent',
-  // CCS's channel enum for mobile push is 'PUSH', but content/delivery keys stay 'mpush*'.
-  PUSH: 'mpushMessageContent',
-  INAPP: 'inAppMessageContent',
-  ANDROID: 'androidMessageContent',
-  IOS: 'iosMessageContent',
-  ZALO: 'zaloMessageContent',
-  VIBER: 'viberMessageContent',
-  LINE: 'lineMessageContent',
-  WEBPUSH: 'webPushMessageContent',
-  RCS: 'rcsMessageContent',
-};
-
-// Transforms legacy messageMeta field names to CCS's before sending, for channels
-// where the two shapes don't already match.
-export const CCS_CONTENT_TRANSFORMS = {
-  EMAIL: (templateData = {}) => ({
-    messageSubject: templateData.emailSubject,
-    messageBody: templateData.emailBody || templateData.emailHtml,
-  }),
-  SMS: (templateData = {}) => ({
-    message: templateData.messageBody || '',
-  }),
-  WEBPUSH: (templateData = {}) => {
-    const inner = templateData.messageContent?.content || templateData;
-    return {
-      messageSubject: inner.messageSubject,
-      accountId: inner.accountId,
-      content: inner.content,
-    };
-  },
-};
-
-export const CCS_CHANNEL_DELIVERY_KEY_MAP = {
-  SMS: 'smsDeliverySettings',
-  EMAIL: 'emailDeliverySettings',
-  WHATSAPP: 'whatsappDeliverySettings',
-  PUSH: 'mpushDeliverySettings',
-  INAPP: 'inAppDeliverySettings',
-  ANDROID: 'androidDeliverySettings',
-  IOS: 'iosDeliverySettings',
-  ZALO: 'zaloDeliverySettings',
-  VIBER: 'viberDeliverySettings',
-  LINE: 'lineDeliverySettings',
-  WEBPUSH: 'webPushDeliverySettings',
-  RCS: 'rcsDeliverySettings',
-};
-
-// UI uses `MOBILEPUSH`; CCS uses `PUSH` for the channel enum. Other channels pass through.
-export const CCS_CHANNEL_NAME_MAP = {
-  MOBILEPUSH: 'PUSH',
-};
-
-// Channel config is the shared source of truth: paneKey maps TemplatesV2 panes, channelProp maps CreativesContainer tabs.
+// Channel config - single source of truth for CreativesContainer and TemplatesV2
+// paneKey: TemplatesV2 defaultPanes object key (for channelsToHide)
+// channelProp: CreativesContainer channel prop (must match pane.key for tab to be active)
 export const CHANNELS = [
   {
     value: 'SMS',
@@ -291,6 +232,4 @@ export const INCENTIVE_TYPES = [
   {
     value: 'badges', label: <FormattedMessage {...messages.badges} />, isActive: false, disabled: false,
   },
-];
-
-export const REFERENCE_ID_EXISTS = 'REFERENCE_ID_EXISTS';
+];

package/v2Containers/CommunicationFlow/index.js

@@ -9,6 +9,7 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import CommunicationFlow from './CommunicationFlow';
+import { hasSupportEngagementModule } from '../../utils/common';
 
 const CommunicationFlowContainer = ({
   config,
@@ -17,20 +18,26 @@ const CommunicationFlowContainer = ({
   onCancel,
   onChange,
   ...otherProps
-}) => (
-  <CommunicationFlow
-    config={config}
-    initialData={initialData}
-    onSave={onSave}
-    onCancel={onCancel}
-    onChange={onChange}
-    {...otherProps}
-  />
-);
+}) => {
+  if (!hasSupportEngagementModule()) {
+    return null;
+  }
+
+  return (
+    <CommunicationFlow
+      config={config}
+      initialData={initialData}
+      onSave={onSave}
+      onCancel={onCancel}
+      onChange={onChange}
+      {...otherProps}
+    />
+  );
+};
 
 CommunicationFlowContainer.propTypes = {
   config: PropTypes.shape({
-    consumer: PropTypes.oneOf(['campaigns', 'loyalty', 'adiona', 'coupons', 'cartPromotions']).isRequired,
+    consumer: PropTypes.oneOf(['campaigns', 'loyalty', 'adiona']).isRequired,
     channel: PropTypes.string, // optional if channel step is skipped
     mode: PropTypes.oneOf(['create', 'edit', 'preview']).isRequired,
     channelsToHide: PropTypes.arrayOf(PropTypes.string),
@@ -80,13 +87,13 @@ CommunicationFlowContainer.propTypes = {
         controls: PropTypes.array,
       }),
     }),
-    context: PropTypes.object, // ouId, campaignId, programId, name, referenceId, description
-    useCCS: PropTypes.bool, // skips the CCS createCommDefinition call on save when false; defaults to true
+    context: PropTypes.object, // ouId, campaignId, programId, etc.
+    useCCS: PropTypes.bool, // If false, skips CCS bulk-claim-approve on save. Defaults to true.
   }).isRequired,
   initialData: PropTypes.object, // for edit/preview mode
-  onSave: PropTypes.func.isRequired, // (data) => void, data.ccsCommDefinition set when CCS create succeeded
-  onCancel: PropTypes.func.isRequired, // () => void
-  onChange: PropTypes.func, // (data) => void
+  onSave: PropTypes.func.isRequired, // (data) => void - called when user saves
+  onCancel: PropTypes.func.isRequired, // () => void - called when user cancels
+  onChange: PropTypes.func, // (data) => void - optional, called on data changes
 };
 
 CommunicationFlowContainer.defaultProps = {

package/v2Containers/CommunicationFlow/messages.js

@@ -64,9 +64,9 @@ export default {
   },
 
   // Slidebox header
-  addContent: {
-    id: `${prefix}.addContent`,
-    defaultMessage: 'Add content',
+  addMessage: {
+    id: `${prefix}.addMessage`,
+    defaultMessage: 'Add message',
   },
 
   // Step prompts
@@ -363,12 +363,4 @@ export default {
     id: `${prefix}.senderNotConfiguredError`,
     defaultMessage: 'Selected domain gateway id is not correct. Please change the domain id or contact the gateway team to register them with Capillary.',
   },
-  duplicateReferenceIdError: {
-    id: `${prefix}.duplicateReferenceIdError`,
-    defaultMessage: 'This reference ID is already in use in your organization. Please use a different reference ID.',
-  },
-  genericSaveError: {
-    id: `${prefix}.genericSaveError`,
-    defaultMessage: 'Something went wrong while saving your content. Please try again.',
-  },
 };

package/v2Containers/CommunicationFlow/steps/ChannelSelectionStep/ChannelSelectionStep.js

@@ -45,7 +45,6 @@ const ChannelSelectionStep = ({
   onChange,
   channelsToHide = [],
   channelsToDisable = [],
-  disablePreviewAndTest = false,
   creativesMode = 'create',
   selectedOfferDetails = [],
   incentivesData,
@@ -64,7 +63,6 @@ const ChannelSelectionStep = ({
   const [showIncentivesMenuMap, setShowIncentivesMenuMap] = useState({});
   const [showTestAndPreview, setShowTestAndPreview] = useState(false);
   const [testAndPreviewItem, setTestAndPreviewItem] = useState(null);
-  const [domainPropertiesData, setDomainPropertiesData] = useState(null);
   const { formatMessage } = intl || {};
 
   // Available channels (filter out hidden ones)
@@ -78,19 +76,6 @@ const ChannelSelectionStep = ({
     const selectedChannelLowerCase = selectedChannel?.toLowerCase();
     return CHANNELS.find((channel) => channel?.value?.toLowerCase() === selectedChannelLowerCase || channel?.channelProp === selectedChannelLowerCase) || null;
   }, [selectedChannel]);
-  
-  const editingZaloHostName = useMemo(() => {
-    if (!editingContentId) return '';
-    const editingItem = contentItems.find((c) => c.contentId === editingContentId);
-    if (editingItem?.channel?.toUpperCase() !== ZALO) return '';
-    const accountId = editingItem?.templateData?.accountId;
-    if (!accountId) return '';
-    const zaloDomains = domainPropertiesData?.ZALO || [];
-    const matchedDomain = zaloDomains.find(
-      (domain) => String(domain?.domainProperties?.connectionProperties?.oa_id) === String(accountId),
-    );
-    return matchedDomain?.domainProperties?.hostName || '';
-  }, [editingContentId, contentItems, domainPropertiesData]);
   /**
    * Handle CreativesContainer close
    */
@@ -373,7 +358,7 @@ const ChannelSelectionStep = ({
               >
                  {formatMessage(messages.edit)}
               </CapMenu.Item>
-              {!disablePreviewAndTest && !PREVIEW_TEST_UNSUPPORTED_CHANNELS.includes(item?.channel?.toUpperCase()) && (
+              {!PREVIEW_TEST_UNSUPPORTED_CHANNELS.includes(item?.channel?.toUpperCase()) && (
                 <CapMenu.Item
                   id="preview-menu-item"
                   className="ant-dropdown-menu-item"
@@ -422,6 +407,10 @@ const ChannelSelectionStep = ({
   return (
     <>
       <CapRow className="channel-selection-step">
+        <CapHeading type="h3" className="margin-b-8">
+          {formatMessage(messages.contentTemplate)}
+        </CapHeading>
+
         <CapRow
           useLegacy
           className={`content-template-container ${contentItems?.length > 0 ? 'content-template-container--has-content' : ''}`}
@@ -433,7 +422,7 @@ const ChannelSelectionStep = ({
           )}
 
           {contentItems?.length === 0 ? (
-            <CapRow className={`content-template-section ${contentItems?.length === 0 ? 'no-content-items' : ''}`}>
+            <CapRow className="content-template-section">
               <CapDropdown
                 overlay={renderChannelDropdownOverlay() || <CapMenu />}
                 onVisibleChange={handleChannelsVisibleChange}
@@ -481,7 +470,6 @@ const ChannelSelectionStep = ({
               deliverySettingsData={deliverySettingsData}
               deliverySetting={value?.deliverySetting}
               onDeliverySettingChange={(deliverySetting) => onChange({ deliverySetting })}
-              onDomainPropertiesLoaded={setDomainPropertiesData}
               intl={intl}
             />
           )}
@@ -502,13 +490,14 @@ const ChannelSelectionStep = ({
           getCreativesData={handleCreativesData}
           handleCloseCreatives={handleCloseCreatives}
           isFullMode={false}
-          hostName={editingZaloHostName}
-          messageDetails={{ type: (config?.context?.module || '').toLowerCase() || 'default' }}
-          location={{ query: { type: 'embedded', module: config?.context?.module } }}
+          messageDetails={{ type: 'default' }}
           templateData={editingContentId ? (() => {
             const saved = contentItems.find((c) => c.contentId === editingContentId)?.templateData;
+            // getTemplateData in CreativesContainer reads 'content', 'accountId', 'messageSubject' at
+            // top-level and needs 'type' for SlideBoxContent to set isEditWebPush. Our stored WEBPUSH
+            // templateData wraps those fields inside messageContent.content, so extract them here.
             if (saved?.channel?.toUpperCase() === WEBPUSH && saved?.messageContent?.content) {
-              return { ...saved.messageContent.content, type: WEBPUSH, channel: WEBPUSH };
+              return { ...saved.messageContent.content, type: WEBPUSH };
             }
             return saved;
           })() : null}
@@ -566,7 +555,6 @@ ChannelSelectionStep.propTypes = {
   onChange: PropTypes.func.isRequired,
   channelsToHide: PropTypes.array,
   channelsToDisable: PropTypes.array,
-  disablePreviewAndTest: PropTypes.bool,
   creativesMode: PropTypes.oneOf(['create', 'edit', 'preview']),
   selectedOfferDetails: PropTypes.array,
   incentivesData: PropTypes.shape({
@@ -588,7 +576,6 @@ ChannelSelectionStep.defaultProps = {
   error: null,
   channelsToHide: [],
   channelsToDisable: [],
-  disablePreviewAndTest: false,
   creativesMode: 'create',
   selectedOfferDetails: [],
   incentivesData: null,

package/v2Containers/CommunicationFlow/steps/ChannelSelectionStep/ChannelSelectionStep.scss

@@ -130,10 +130,6 @@
     border-bottom: 1px solid $CAP_G07;
   }
 
-  .no-content-items {
-    border-bottom: none;
-  }
-
   .add-content-template-button {
     padding: $CAP_SPACE_12 $CAP_SPACE_16;
     align-self: center !important;

package/v2Containers/CommunicationFlow/steps/ChannelSelectionStep/Tests/ChannelSelectionStep.test.js

@@ -37,21 +37,9 @@ jest.mock('../../../../CreativesContainer', () => function MockCreativesContaine
   handleCloseCreatives,
   creativesMode,
   channel,
-  templateData,
-  hostName,
-  messageDetails,
-  location,
 }) {
   return (
-    <div
-      data-testid="creatives-mock"
-      data-creatives-mode={creativesMode}
-      data-creatives-channel={channel}
-      data-template-data={JSON.stringify(templateData)}
-      data-host-name={hostName}
-      data-message-details={JSON.stringify(messageDetails)}
-      data-location={JSON.stringify(location)}
-    >
+    <div data-testid="creatives-mock" data-creatives-mode={creativesMode} data-creatives-channel={channel}>
       <button
         type="button"
         data-testid="creatives-save"
@@ -74,7 +62,7 @@ jest.mock('../../../../CreativesContainer', () => function MockCreativesContaine
 });
 
 jest.mock('../../DeliverySettingsStep', () => ({
-  DeliverySettingsSection: function MockDeliverySettings({ onDeliverySettingChange, onDomainPropertiesLoaded }) {
+  DeliverySettingsSection: function MockDeliverySettings({ onDeliverySettingChange }) {
     return (
       <div data-testid="delivery-settings-section">
         <button
@@ -84,23 +72,6 @@ jest.mock('../../DeliverySettingsStep', () => ({
         >
           Apply delivery
         </button>
-        <button
-          type="button"
-          data-testid="domain-properties-loaded"
-          onClick={() => onDomainPropertiesLoaded?.({
-            ZALO: [{
-              id: 267284,
-              domainProperties: {
-                id: 4977,
-                domainName: 'Gapit_Automation',
-                connectionProperties: { oa_id: '300086756699856746' },
-                hostName: 'gapitzalotrans',
-              },
-            }],
-          })}
-        >
-          Load domain properties
-        </button>
       </div>
     );
   },
@@ -123,7 +94,7 @@ describe('ChannelSelectionStep', () => {
     console.log.mockRestore();
   });
 
-  it('empty state: shows the Channel heading and add-template entry point', () => {
+  it('empty state: shows content template heading and add-template entry point', () => {
     const onChange = jest.fn();
     renderStep(
       <ChannelSelectionStep
@@ -133,7 +104,7 @@ describe('ChannelSelectionStep', () => {
       />,
     );
 
-    expect(screen.getAllByText('Channel').length).toBeGreaterThanOrEqual(1);
+    expect(screen.getAllByText('Content template').length).toBeGreaterThanOrEqual(1);
     expect(screen.getByRole('button', { name: /add creative/i })).toBeInTheDocument();
     expect(screen.queryByTestId('creatives-mock')).not.toBeInTheDocument();
   });
@@ -1712,149 +1683,6 @@ describe('ChannelSelectionStep', () => {
     expect(screen.getByTestId('creatives-mock')).toHaveAttribute('data-creatives-mode', 'edit');
   });
 
-  it('edit WEBPUSH item includes `channel: WEBPUSH` (not just `type`) in the unwrapped templateData (regression: CreativesContainer.getTemplateData\'s switch reads templateData.channel, not templateData.type — without it the switch never matched WEBPUSH and the edit screen rendered blank below its header)', async () => {
-    renderStep(
-      <ChannelSelectionStep
-        value={{
-          contentItems: [{
-            contentId: 'wp-edit-channel',
-            channel: 'WEBPUSH',
-            templateData: {
-              channel: 'WEBPUSH',
-              messageContent: {
-                content: { messageSubject: 'dasd', accountId: 13792, content: { title: 'dasd', message: 'dasd' } },
-              },
-            },
-          }],
-        }}
-        onChange={jest.fn()}
-        channels={CHANNELS}
-      />,
-    );
-    await userEvent.click(screen.getByLabelText('Show more content options icon'));
-    await waitFor(() => expect(screen.getByRole('menu')).toBeInTheDocument(), WAIT_OPTIONS);
-    await userEvent.click(within(screen.getByRole('menu')).getByText('Edit'));
-    await waitFor(() => expect(screen.getByTestId('creatives-mock')).toBeInTheDocument(), WAIT_OPTIONS);
-    const passedTemplateData = JSON.parse(screen.getByTestId('creatives-mock').getAttribute('data-template-data'));
-    expect(passedTemplateData.channel).toBe('WEBPUSH');
-    expect(passedTemplateData.type).toBe('WEBPUSH');
-    expect(passedTemplateData.messageSubject).toBe('dasd');
-    expect(passedTemplateData.accountId).toBe(13792);
-  });
-
-  // ── ZALO edit: hostName resolved from domainProperties ────────────────────────
-
-  it('resolves a Zalo item\'s hostName from the domainProperties fetch by matching accountId to connectionProperties.oa_id, and passes it to CreativesContainer (regression: CCS\'s zaloMessageContent never carries hostName, and without it Zalo/index.js\'s getTemplateInfoById guard never passes, so the edit view never loads live template data)', async () => {
-    renderStep(
-      <ChannelSelectionStep
-        value={{
-          contentItems: [{
-            contentId: 'zalo-edit',
-            channel: 'ZALO',
-            templateData: {
-              channel: 'ZALO',
-              accountId: '300086756699856746',
-              accountName: 'gapit_automation_account',
-              token: 'zalo-token',
-              templateConfigs: { id: '630142', name: '1592_Chí Linh_D1' },
-            },
-          }],
-        }}
-        onChange={jest.fn()}
-        channels={CHANNELS}
-        deliverySettingsData={{ required: false }}
-      />,
-    );
-    await userEvent.click(screen.getByTestId('domain-properties-loaded'));
-    await userEvent.click(screen.getByLabelText('Show more content options icon'));
-    await waitFor(() => expect(screen.getByRole('menu')).toBeInTheDocument(), WAIT_OPTIONS);
-    await userEvent.click(within(screen.getByRole('menu')).getByText('Edit'));
-    await waitFor(() => expect(screen.getByTestId('creatives-mock')).toBeInTheDocument(), WAIT_OPTIONS);
-    expect(screen.getByTestId('creatives-mock')).toHaveAttribute('data-host-name', 'gapitzalotrans');
-  });
-
-  it('resolves an empty hostName for a Zalo item when no matching domainProperties account is loaded', async () => {
-    renderStep(
-      <ChannelSelectionStep
-        value={{
-          contentItems: [{
-            contentId: 'zalo-edit-no-match',
-            channel: 'ZALO',
-            templateData: { channel: 'ZALO', accountId: 'unmatched-account-id' },
-          }],
-        }}
-        onChange={jest.fn()}
-        channels={CHANNELS}
-        deliverySettingsData={{ required: false }}
-      />,
-    );
-    await userEvent.click(screen.getByLabelText('Show more content options icon'));
-    await waitFor(() => expect(screen.getByRole('menu')).toBeInTheDocument(), WAIT_OPTIONS);
-    await userEvent.click(within(screen.getByRole('menu')).getByText('Edit'));
-    await waitFor(() => expect(screen.getByTestId('creatives-mock')).toBeInTheDocument(), WAIT_OPTIONS);
-    expect(screen.getByTestId('creatives-mock')).toHaveAttribute('data-host-name', '');
-  });
-
-  // ── Tag-context pass-through (config.context.module) ──────────────────────────
-
-  it('passes config.context.module through to messageDetails.type (lowercased) and location.query.module, unchanged, for tag-fetch scoping (regression: this used to be hardcoded to messageDetails={{type: \'default\'}} with no location prop at all, so every consumer\'s tag-fetch was scoped generically instead of per-consumer)', async () => {
-    renderStep(
-      <ChannelSelectionStep
-        value={{ contentItems: [] }}
-        onChange={jest.fn()}
-        channels={CHANNELS}
-        config={{ context: { module: 'CAMPAIGNS' } }}
-      />,
-    );
-    await userEvent.click(screen.getByRole('button', { name: /add creative/i }));
-    await waitFor(() => expect(screen.getByRole('menu')).toBeInTheDocument(), WAIT_OPTIONS);
-    await userEvent.click(within(screen.getByRole('menu')).getByText('SMS'));
-    await waitFor(() => expect(screen.getByTestId('creatives-mock')).toBeInTheDocument(), WAIT_OPTIONS);
-
-    const mock = screen.getByTestId('creatives-mock');
-    expect(JSON.parse(mock.getAttribute('data-message-details'))).toEqual({ type: 'campaigns' });
-    expect(JSON.parse(mock.getAttribute('data-location'))).toEqual({
-      query: { type: 'embedded', module: 'CAMPAIGNS' },
-    });
-  });
-
-  it('passes through whatever module value a coupons/cartPromotions-style config sets, with no special-casing on our end', async () => {
-    renderStep(
-      <ChannelSelectionStep
-        value={{ contentItems: [] }}
-        onChange={jest.fn()}
-        channels={CHANNELS}
-        config={{ context: { module: 'coupons' } }}
-      />,
-    );
-    await userEvent.click(screen.getByRole('button', { name: /add creative/i }));
-    await waitFor(() => expect(screen.getByRole('menu')).toBeInTheDocument(), WAIT_OPTIONS);
-    await userEvent.click(within(screen.getByRole('menu')).getByText('SMS'));
-    await waitFor(() => expect(screen.getByTestId('creatives-mock')).toBeInTheDocument(), WAIT_OPTIONS);
-
-    const mock = screen.getByTestId('creatives-mock');
-    expect(JSON.parse(mock.getAttribute('data-message-details'))).toEqual({ type: 'coupons' });
-    expect(JSON.parse(mock.getAttribute('data-location'))).toEqual({
-      query: { type: 'embedded', module: 'coupons' },
-    });
-  });
-
-  it('falls back to messageDetails={{type: "default"}} when config.context.module is absent (unset consumers keep prior behavior)', async () => {
-    renderStep(
-      <ChannelSelectionStep
-        value={{ contentItems: [] }}
-        onChange={jest.fn()}
-        channels={CHANNELS}
-      />,
-    );
-    await userEvent.click(screen.getByRole('button', { name: /add creative/i }));
-    await waitFor(() => expect(screen.getByRole('menu')).toBeInTheDocument(), WAIT_OPTIONS);
-    await userEvent.click(within(screen.getByRole('menu')).getByText('SMS'));
-    await waitFor(() => expect(screen.getByTestId('creatives-mock')).toBeInTheDocument(), WAIT_OPTIONS);
-
-    expect(JSON.parse(screen.getByTestId('creatives-mock').getAttribute('data-message-details'))).toEqual({ type: 'default' });
-  });
-
   // ── FTP channel filtered from dropdown ───────────────────────────────────────
 
   it('FTP channel is filtered out of the channel dropdown', async () => {

package/v2Containers/CommunicationFlow/steps/DeliverySettingsStep/DeliverySettingsSection.js

@@ -33,7 +33,6 @@ const DeliverySettingsSection = ({
   deliverySettingsData,
   deliverySetting = {},
   onDeliverySettingChange,
-  onDomainPropertiesLoaded,
   intl,
 }) => {
   const [showSlidebox, setShowSlidebox] = useState(false);
@@ -103,7 +102,6 @@ const DeliverySettingsSection = ({
             )
             : raw;
           setDomainPropertiesData(entity);
-          onDomainPropertiesLoaded?.(entity);
         }
       } catch (err) {
         if (!cancelled) setDomainPropertiesData(null);
@@ -318,7 +316,6 @@ DeliverySettingsSection.propTypes = {
   deliverySettingsData: PropTypes.object,
   deliverySetting: PropTypes.object,
   onDeliverySettingChange: PropTypes.func,
-  onDomainPropertiesLoaded: PropTypes.func,
   intl: PropTypes.object.isRequired,
 };
 

package/v2Containers/CommunicationFlow/steps/DeliverySettingsStep/SenderDetails.js

@@ -29,7 +29,6 @@ import {
   getEmailDefaultsForDomainId,
   getSmsDefaultsForDomainId,
   getRcsAccountName,
-  getEmailDomainGatewayMapIdByDomainId,
 } from './deliverySettingsConfig';
 import messages from '../../messages';
 import './SenderDetails.scss';
@@ -152,11 +151,6 @@ const SenderDetails = ({
           saveValues.whatsappDomainId = matchedDomain?.domainProperties?.id ?? matchedDomain?.domainId ?? matchedDomain?.id;
         }
       }
-      // Include EMAIL domainGatewayMapId derived from the selected domain for persistence -
-      // CCS needs it alongside domainId, but it's not a user-facing field.
-      if (entity && fieldValues.emailDomain != null && fieldValues.emailDomain !== '') {
-        saveValues.emailDomainGatewayMapId = getEmailDomainGatewayMapIdByDomainId(entity, fieldValues.emailDomain);
-      }
       onSave(saveValues);
     }
     onClose();