@capillarytech/creatives-library 9.0.50-alpha.0 → 9.0.50-alpha.1

package/v2Containers/CommunicationFlow/CommunicationFlowCard.js

@@ -19,7 +19,6 @@ import CapIcon from '@capillarytech/cap-ui-library/CapIcon';
 import CapLabel from '@capillarytech/cap-ui-library/CapLabel';
 import CapHeader from '@capillarytech/cap-ui-library/CapHeader';
 import CapSlideBox from '@capillarytech/cap-ui-library/CapSlideBox';
-import CapTooltip from '@capillarytech/cap-ui-library/CapTooltip';
 import CommunicationFlow from './index';
 import { CHANNELS, DEFAULT_COMMUNICATION_STRATEGY_OPTIONS, DYNAMIC_CONTROLS_CONFIG } from './constants';
 import {
@@ -75,8 +74,6 @@ const CommunicationFlowCard = ({
   onChange,
   cap,
   intl,
-  disabled,
-  disabledTooltip,
 }) => {
   const { formatMessage } = intl || {};
   const [showSlideBox, setShowSlideBox] = useState(false);
@@ -94,9 +91,8 @@ const CommunicationFlowCard = ({
   }, [onCancel]);
 
   const handleOpen = useCallback(() => {
-    if (disabled) return;
     setShowSlideBox(true);
-  }, [disabled]);
+  }, []);
 
   const firstItem = savedData?.contentItems?.[0];
   const channelConfig = firstItem
@@ -186,17 +182,9 @@ const CommunicationFlowCard = ({
               <CapImage src={addCreativesIllustration} />
             </CapColumn>
             <CapColumn span={14} className="empty-card-action-col">
-              {disabled && disabledTooltip ? (
-                <CapTooltip title={disabledTooltip}>
-                  <CapButton type="secondary" disabled>
-                    {formatMessage(messages.addCreatives)}
-                  </CapButton>
-                </CapTooltip>
-              ) : (
-                <CapButton type="secondary" onClick={handleOpen} disabled={disabled}>
-                  {formatMessage(messages.addCreatives)}
-                </CapButton>
-              )}
+              <CapButton type="secondary" onClick={handleOpen}>
+                {formatMessage(messages.addCreatives)}
+              </CapButton>
             </CapColumn>
           </CapRow>
         </CapCard>
@@ -239,8 +227,6 @@ CommunicationFlowCard.propTypes = {
   onChange: PropTypes.func,
   cap: PropTypes.object,
   intl: PropTypes.object.isRequired,
-  disabled: PropTypes.bool, // Disables the "Add creatives" trigger (e.g. until a consumer-required field is filled)
-  disabledTooltip: PropTypes.node, // Shown on hover when disabled is true
 };
 
 CommunicationFlowCard.defaultProps = {
@@ -249,8 +235,6 @@ CommunicationFlowCard.defaultProps = {
   onCancel: null,
   onChange: null,
   cap: null,
-  disabled: false,
-  disabledTooltip: null,
 };
 
 export default injectIntl(CommunicationFlowCard);

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

@@ -1,7 +1,8 @@
 import React from 'react';
 
 jest.mock('../../../services/api', () => ({
-  createCommDefinition: jest.fn(),
+  createCentralCommsMetaId: jest.fn(),
+  getCentralCommsMetaIds: jest.fn(),
 }));
 
 jest.mock('../../CreativesContainer', () => function MockCreativesContainer({
@@ -37,7 +38,7 @@ import { IntlProvider } from 'react-intl';
 import history from '../../../utils/history';
 import { initialReducer } from '../../../initialReducer';
 import CommunicationFlow from '../CommunicationFlow';
-import { createCommDefinition } from '../../../services/api';
+import { createCentralCommsMetaId, getCentralCommsMetaIds } from '../../../services/api';
 import { getEnabledSteps } from '../utils/getEnabledSteps';
 import {
   CHANNELS,
@@ -358,93 +359,74 @@ describe('isSaveDisabled', () => {
 });
 
 describe('handleSave — CCS flow', () => {
-  const ccsConfig = { ...baseConfig, context: { name: 'Order Placed Notification' }, features: {} };
-
   beforeEach(() => {
-    createCommDefinition.mockResolvedValue({
-      response: { data: { id: 'cd_123', referenceId: 'ORDER_PLACED_123', status: 'DRAFT', version: { version: 1 } } },
-    });
+    createCentralCommsMetaId.mockResolvedValue({ response: { data: { id: 'meta-123' } } });
+    getCentralCommsMetaIds.mockResolvedValue({ response: { data: {} } });
   });
 
   afterEach(() => {
     jest.clearAllMocks();
   });
 
-  it('calls createCommDefinition with a SINGLE-strategy payload when useCCS is not false', async () => {
+  it('calls createCentralCommsMetaId for each content item when useCCS is not false', async () => {
     const onSave = jest.fn();
     renderWithFlow({
       features: {},
-      config: ccsConfig,
       initialData: {
-        contentItems: [{ channel: 'sms', templateData: { message: 'Hello' } }],
+        contentItems: [{ channel: 'sms', templateData: { smsBody: 'Hello' } }],
       },
       onSave,
     });
 
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
-    await waitFor(() => expect(createCommDefinition).toHaveBeenCalledTimes(1));
-    expect(createCommDefinition).toHaveBeenCalledWith(
+    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalledTimes(1));
+    expect(createCentralCommsMetaId).toHaveBeenCalledWith(
       expect.objectContaining({
-        name: 'Order Placed Notification',
-        strategyType: 'SINGLE',
-        singleChannelStrategy: expect.objectContaining({
-          channel: 'SMS',
-          smsMessageContent: { message: 'Hello' },
-        }),
+        centralCommsPayload: expect.objectContaining({ channel: 'SMS', module: 'CAMPAIGNS' }),
       }),
     );
     expect(onSave).toHaveBeenCalledTimes(1);
   });
 
-  it('merges ccsCommDefinition into the data passed to onSave when create succeeds', async () => {
-    const onSave = jest.fn();
+  it('calls getCentralCommsMetaIds when metaIds are returned from createCentralCommsMetaId', async () => {
     renderWithFlow({
       features: {},
-      config: ccsConfig,
-      initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
-      onSave,
+      initialData: { contentItems: [{ channel: 'EMAIL', templateData: {} }] },
     });
 
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
-    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(onSave).toHaveBeenCalledWith(
-      expect.objectContaining({
-        ccsCommDefinition: { id: 'cd_123', referenceId: 'ORDER_PLACED_123', version: 1, status: 'DRAFT' },
-      }),
-    );
+    await waitFor(() => expect(getCentralCommsMetaIds).toHaveBeenCalledWith('meta-123'));
   });
 
-  it('generates a referenceId from name when config.context.referenceId is absent', async () => {
-    // moduleMocks.js spies on Date.now globally, but this config's resetMocks:true
-    // clears that return value before every test — set it explicitly here.
-    jest.spyOn(Date, 'now').mockReturnValue(1612267539410);
+  it('skips getCentralCommsMetaIds when response contains no id', async () => {
+    createCentralCommsMetaId.mockResolvedValue({ response: { data: {} } });
     renderWithFlow({
       features: {},
-      config: ccsConfig,
       initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
     });
 
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
-    await waitFor(() => expect(createCommDefinition).toHaveBeenCalled());
-    const payload = createCommDefinition.mock.calls[0][0];
-    expect(payload.referenceId).toBe('ORDER_PLACED_NOTIFICATION_1612267539410');
+    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalled());
+    expect(getCentralCommsMetaIds).not.toHaveBeenCalled();
   });
 
-  it('uses config.context.referenceId and description when provided', async () => {
+  it('uses ouId and module from config.context when provided', async () => {
     renderWithFlow({
       features: {},
-      config: { ...ccsConfig, context: { ...ccsConfig.context, referenceId: 'ORDER_PLACED', description: 'desc' } },
+      config: { ...baseConfig, context: { ouId: 42, module: 'LOYALTY' }, features: {} },
       initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
     });
 
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
-    await waitFor(() => expect(createCommDefinition).toHaveBeenCalled());
-    expect(createCommDefinition).toHaveBeenCalledWith(
-      expect.objectContaining({ referenceId: 'ORDER_PLACED', description: 'desc' }),
+    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalled());
+    expect(createCentralCommsMetaId).toHaveBeenCalledWith(
+      expect.objectContaining({
+        centralCommsPayload: expect.objectContaining({ ouId: 42, module: 'LOYALTY' }),
+      }),
     );
   });
 
@@ -452,7 +434,7 @@ describe('handleSave — CCS flow', () => {
     const onSave = jest.fn();
     renderWithFlow({
       features: {},
-      config: { ...ccsConfig, useCCS: false },
+      config: { ...baseConfig, useCCS: false, features: {} },
       initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
       onSave,
     });
@@ -460,50 +442,14 @@ describe('handleSave — CCS flow', () => {
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
     await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(createCommDefinition).not.toHaveBeenCalled();
-    expect(onSave).toHaveBeenCalledWith(expect.not.objectContaining({ ccsCommDefinition: expect.anything() }));
-  });
-
-  it('skips createCommDefinition when config.context.name is absent (mandatory)', async () => {
-    const onSave = jest.fn();
-    renderWithFlow({
-      features: {},
-      config: { ...baseConfig, features: {} },
-      initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
-      onSave,
-    });
-
-    await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
-
-    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(createCommDefinition).not.toHaveBeenCalled();
-  });
-
-  it('skips createCommDefinition for multi-channel strategies (CHANNEL_PRIORITY/AB_TEST)', async () => {
-    const onSave = jest.fn();
-    renderWithFlow({
-      features: {},
-      config: ccsConfig,
-      initialData: {
-        communicationStrategy: CHANNEL_PRIORITY,
-        channels: ['SMS', 'EMAIL'],
-        contentItems: [{ channel: 'SMS', templateData: {} }, { channel: 'EMAIL', templateData: {} }],
-      },
-      onSave,
-    });
-
-    await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
-
-    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(createCommDefinition).not.toHaveBeenCalled();
+    expect(createCentralCommsMetaId).not.toHaveBeenCalled();
   });
 
-  it('still calls onSave (without ccsCommDefinition) when createCommDefinition rejects', async () => {
-    createCommDefinition.mockRejectedValue(new Error('Network error'));
+  it('still calls onSave when createCentralCommsMetaId rejects', async () => {
+    createCentralCommsMetaId.mockRejectedValue(new Error('Network error'));
     const onSave = jest.fn();
     renderWithFlow({
       features: {},
-      config: ccsConfig,
       initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
       onSave,
     });
@@ -511,35 +457,12 @@ describe('handleSave — CCS flow', () => {
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
     await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(onSave).toHaveBeenCalledWith(expect.not.objectContaining({ ccsCommDefinition: expect.anything() }));
   });
 
-  it('blocks save and shows an error when createCommDefinition resolves with a duplicate REFERENCE_ID_EXISTS conflict', async () => {
-    createCommDefinition.mockResolvedValue({
-      success: false,
-      status: { isError: true, code: 409, message: 'REFERENCE_ID_EXISTS' },
-      message: 'REFERENCE_ID_EXISTS',
-    });
+  it('skips createCentralCommsMetaId when contentItems is empty', async () => {
     const onSave = jest.fn();
     renderWithFlow({
       features: {},
-      config: ccsConfig,
-      initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
-      onSave,
-    });
-
-    await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
-
-    await waitFor(() => expect(createCommDefinition).toHaveBeenCalledTimes(1));
-    expect(await screen.findByText(/already in use in your organization/i)).toBeInTheDocument();
-    expect(onSave).not.toHaveBeenCalled();
-  });
-
-  it('skips createCommDefinition when contentItems is empty', async () => {
-    const onSave = jest.fn();
-    renderWithFlow({
-      features: {},
-      config: ccsConfig,
       initialData: { contentItems: [] },
       onSave,
     });
@@ -547,13 +470,12 @@ describe('handleSave — CCS flow', () => {
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
     await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(createCommDefinition).not.toHaveBeenCalled();
+    expect(createCentralCommsMetaId).not.toHaveBeenCalled();
   });
 
-  it('places additionalSettings derived from dynamicControls under settings.additionalSettings (not delivery settings)', async () => {
+  it('includes additionalSettings derived from dynamicControls in the payload', async () => {
     renderWithFlow({
       features: {},
-      config: ccsConfig,
       initialData: {
         contentItems: [{ channel: 'SMS', templateData: {} }],
         dynamicControls: {
@@ -567,36 +489,14 @@ describe('handleSave — CCS flow', () => {
 
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
-    await waitFor(() => expect(createCommDefinition).toHaveBeenCalled());
-    const payload = createCommDefinition.mock.calls[0][0];
-    expect(payload.settings.additionalSettings).toEqual({
+    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalled());
+    const payload = createCentralCommsMetaId.mock.calls[0][0];
+    expect(payload.centralCommsPayload.smsDeliverySettings.additionalSettings).toEqual({
       useTinyUrl: true,
       encryptUrl: true,
       linkTrackingEnabled: true,
       userSubscriptionDisabled: true,
     });
-    expect(payload.singleChannelStrategy.smsDeliverySettings.additionalSettings).toBeUndefined();
-  });
-
-  it('includes channelSettings from deliverySetting.channelSetting in the payload', async () => {
-    renderWithFlow({
-      features: {},
-      config: ccsConfig,
-      initialData: {
-        contentItems: [{ channel: 'SMS', templateData: {} }],
-        deliverySetting: { channelSetting: { SMS: { domainId: 1001, gsmSenderId: 'BRAND1' } } },
-      },
-    });
-
-    await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
-
-    await waitFor(() => expect(createCommDefinition).toHaveBeenCalled());
-    const payload = createCommDefinition.mock.calls[0][0];
-    expect(payload.singleChannelStrategy.smsDeliverySettings.channelSettings).toEqual({
-      channel: 'SMS',
-      domainId: 1001,
-      gsmSenderId: 'BRAND1',
-    });
   });
 });
 
@@ -652,34 +552,39 @@ describe('optional chaining safety', () => {
     jest.clearAllMocks();
   });
 
-  it('does not crash and skips CCS when config.context is entirely absent', async () => {
-    const onSave = jest.fn();
+  it('defaults ouId to -1 when config.context is absent', async () => {
+    createCentralCommsMetaId.mockResolvedValue({ response: { data: { id: 'x' } } });
     renderWithFlow({
       features: {},
-      config: { ...baseConfig, features: {} },
       initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
-      onSave,
     });
 
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
-    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(createCommDefinition).not.toHaveBeenCalled();
+    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalled());
+    expect(createCentralCommsMetaId).toHaveBeenCalledWith(
+      expect.objectContaining({
+        centralCommsPayload: expect.objectContaining({ ouId: -1 }),
+      }),
+    );
   });
 
-  it('does not crash when config.context is present but empty', async () => {
-    const onSave = jest.fn();
+  it('defaults module to consumer.toUpperCase() when config.context.module absent', async () => {
+    createCentralCommsMetaId.mockResolvedValue({ response: { data: { id: 'x' } } });
     renderWithFlow({
       features: {},
-      config: { ...baseConfig, context: {}, features: {} },
+      config: { ...baseConfig, consumer: 'loyalty', features: {} },
       initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
-      onSave,
     });
 
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
 
-    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(createCommDefinition).not.toHaveBeenCalled();
+    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalled());
+    expect(createCentralCommsMetaId).toHaveBeenCalledWith(
+      expect.objectContaining({
+        centralCommsPayload: expect.objectContaining({ module: 'LOYALTY' }),
+      }),
+    );
   });
 
   it('initializes channel from config.channel when initialData.channel is absent', () => {

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)', () => {

package/v2Containers/CommunicationFlow/constants.js

@@ -38,46 +38,6 @@ export const CHANNEL_PRIORITY = 'CHANNEL_PRIORITY';
 export const AB_TEST = 'AB_TEST';
 export const SINGLE_TEMPLATE = 'SINGLE_TEMPLATE';
 
-// CCS CommDefinition strategyType — distinct from SINGLE_TEMPLATE above, which
-// is this UI's own communicationStrategy value. Only SINGLE is supported this
-// phase; CCS create is skipped entirely for CHANNEL_PRIORITY/AB_TEST.
-export const CCS_STRATEGY_TYPE_SINGLE = 'SINGLE';
-
-// Mirrors CCS's SingleChannelStrategy field names exactly (Cap Notify API
-// Design doc, "Channel Strategy overview" / class dump). Kept separate from
-// CHANNEL_CONTENT_KEY_MAP/CHANNEL_DELIVERY_KEY_MAP below, which are the legacy
-// messageMeta payload's keys and do not match CCS naming for every channel
-// (e.g. legacy uses lowercase 'mpushMessageContent'; CCS uses 'mPushMessageContent').
-export const CCS_CHANNEL_CONTENT_KEY_MAP = {
-  SMS: 'smsMessageContent',
-  EMAIL: 'emailMessageContent',
-  WHATSAPP: 'whatsappMessageContent',
-  MPUSH: 'mPushMessageContent',
-  INAPP: 'inAppMessageContent',
-  ANDROID: 'androidMessageContent',
-  IOS: 'iosMessageContent',
-  ZALO: 'zaloMessageContent',
-  VIBER: 'viberMessageContent',
-  LINE: 'lineMessageContent',
-  WEBPUSH: 'webPushMessageContent',
-  RCS: 'rcsMessageContent',
-};
-
-export const CCS_CHANNEL_DELIVERY_KEY_MAP = {
-  SMS: 'smsDeliverySettings',
-  EMAIL: 'emailDeliverySettings',
-  WHATSAPP: 'whatsappDeliverySettings',
-  MPUSH: 'mPushDeliverySettings',
-  INAPP: 'inAppDeliverySettings',
-  ANDROID: 'androidDeliverySettings',
-  IOS: 'iosDeliverySettings',
-  ZALO: 'zaloDeliverySettings',
-  VIBER: 'viberDeliverySettings',
-  LINE: 'lineDeliverySettings',
-  WEBPUSH: 'webPushDeliverySettings',
-  RCS: 'rcsDeliverySettings',
-};
-
 // 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)

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,16 +18,22 @@ 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({
@@ -80,13 +87,11 @@ CommunicationFlowContainer.propTypes = {
         controls: PropTypes.array,
       }),
     }),
-    context: PropTypes.object, // ouId, campaignId, programId; name (required for CCS create),
-    // referenceId (optional, auto-generated from name if absent), description (optional).
-    useCCS: PropTypes.bool, // If false, skips the CCS createCommDefinition call on save. 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 - called when user saves; data.ccsCommDefinition
-  // ({id, referenceId, version, status}) is present when the CCS create succeeded
+  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
 };

package/v2Containers/CommunicationFlow/messages.js

@@ -363,8 +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.',
-  },
 };