npm - @capillarytech/creatives-library - Versions diffs - 9.0.54-0 → 9.0.54-alpha.1 - Mend

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

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (19) hide show

package/constants/unified.js +0 -1
package/package.json +1 -1
package/services/api.js +9 -0
package/utils/common.js +0 -4
package/v2Components/CommonTestAndPreview/index.js +0 -3
package/v2Containers/CommunicationFlow/CommunicationFlow.js +96 -58
package/v2Containers/CommunicationFlow/CommunicationFlowCard.js +20 -4
package/v2Containers/CommunicationFlow/Tests/CommunicationFlow.test.js +227 -49
package/v2Containers/CommunicationFlow/Tests/CommunicationFlowCard.test.js +16 -0
package/v2Containers/CommunicationFlow/constants.js +66 -0
package/v2Containers/CommunicationFlow/index.js +15 -20
package/v2Containers/CommunicationFlow/messages.js +4 -0
package/v2Containers/CreativesContainer/index.js +0 -2
package/v2Containers/Rcs/constants.js +0 -1
package/v2Containers/Rcs/index.js +1 -9
package/v2Containers/Rcs/tests/__snapshots__/index.test.js.snap +0 -108
package/v2Containers/Rcs/tests/index.test.js +0 -121
package/v2Containers/Templates/index.js +0 -20
package/.vscode/settings.json +0 -3

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

@@ -1,8 +1,7 @@
 import React from 'react';
 jest.mock('../../../services/api', () => ({
-  createCentralCommsMetaId: jest.fn(),
-  getCentralCommsMetaIds: jest.fn(),
+  createCommDefinition: jest.fn(),
 }));
 jest.mock('../../CreativesContainer', () => function MockCreativesContainer({
@@ -38,7 +37,7 @@ import { IntlProvider } from 'react-intl';
 import history from '../../../utils/history';
 import { initialReducer } from '../../../initialReducer';
 import CommunicationFlow from '../CommunicationFlow';
-import { createCentralCommsMetaId, getCentralCommsMetaIds } from '../../../services/api';
+import { createCommDefinition } from '../../../services/api';
 import { getEnabledSteps } from '../utils/getEnabledSteps';
 import {
   CHANNELS,
@@ -359,82 +358,200 @@ describe('isSaveDisabled', () => {
 });
 describe('handleSave — CCS flow', () => {
+  const ccsConfig = { ...baseConfig, context: { name: 'Order Placed Notification' }, features: {} };
   beforeEach(() => {
-    createCentralCommsMetaId.mockResolvedValue({ response: { data: { id: 'meta-123' } } });
-    getCentralCommsMetaIds.mockResolvedValue({ response: { data: {} } });
+    createCommDefinition.mockResolvedValue({
+      response: { data: { id: 'cd_123', referenceId: 'ORDER_PLACED_123', status: 'DRAFT', version: { version: 1 } } },
+    });
   });
   afterEach(() => {
     jest.clearAllMocks();
   });
-  it('calls createCentralCommsMetaId for each content item when useCCS is not false', async () => {
+  it('calls createCommDefinition with a SINGLE-strategy payload when useCCS is not false', async () => {
     const onSave = jest.fn();
     renderWithFlow({
       features: {},
+      config: ccsConfig,
       initialData: {
-        contentItems: [{ channel: 'sms', templateData: { smsBody: 'Hello' } }],
+        contentItems: [{ channel: 'sms', templateData: { message: 'Hello' } }],
       },
       onSave,
     });
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
-    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalledTimes(1));
-    expect(createCentralCommsMetaId).toHaveBeenCalledWith(
+    await waitFor(() => expect(createCommDefinition).toHaveBeenCalledTimes(1));
+    expect(createCommDefinition).toHaveBeenCalledWith(
       expect.objectContaining({
-        centralCommsPayload: expect.objectContaining({ channel: 'SMS', module: 'CAMPAIGNS' }),
+        name: 'Order Placed Notification',
+        strategyType: 'SINGLE',
+        singleChannelStrategy: expect.objectContaining({
+          variant: expect.objectContaining({
+            channel: 'SMS',
+            smsMessageContent: { channel: 'SMS', message: 'Hello' },
+          }),
+        }),
       }),
     );
     expect(onSave).toHaveBeenCalledTimes(1);
   });
-  it('calls getCentralCommsMetaIds when metaIds are returned from createCentralCommsMetaId', async () => {
+  it('transforms EMAIL content from the legacy editor shape (emailSubject) to CCS\'s messageSubject', async () => {
+    const onSave = jest.fn();
     renderWithFlow({
       features: {},
-      initialData: { contentItems: [{ channel: 'EMAIL', templateData: {} }] },
+      config: ccsConfig,
+      initialData: {
+        contentItems: [{
+          channel: 'EMAIL',
+          templateData: { emailSubject: 'Order Confirmed', messageBody: '<html>Hi</html>' },
+        }],
+      },
+      onSave,
     });
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
-    await waitFor(() => expect(getCentralCommsMetaIds).toHaveBeenCalledWith('meta-123'));
+    await waitFor(() => expect(createCommDefinition).toHaveBeenCalledTimes(1));
+    expect(createCommDefinition).toHaveBeenCalledWith(
+      expect.objectContaining({
+        singleChannelStrategy: expect.objectContaining({
+          variant: expect.objectContaining({
+            channel: 'EMAIL',
+            emailMessageContent: {
+              channel: 'EMAIL',
+              messageSubject: 'Order Confirmed',
+              messageBody: '<html>Hi</html>',
+            },
+          }),
+        }),
+      }),
+    );
   });
-  it('skips getCentralCommsMetaIds when response contains no id', async () => {
-    createCentralCommsMetaId.mockResolvedValue({ response: { data: {} } });
+  it('normalizes the internal MOBILEPUSH channel to CCS\'s PUSH enum (keys stay mpush*)', async () => {
+    const onSave = jest.fn();
     renderWithFlow({
       features: {},
+      config: ccsConfig,
+      initialData: {
+        contentItems: [{ channel: 'MOBILEPUSH', templateData: { messageSubject: 'Hi' } }],
+      },
+      onSave,
+    });
+    await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
+    await waitFor(() => expect(createCommDefinition).toHaveBeenCalledTimes(1));
+    expect(createCommDefinition).toHaveBeenCalledWith(
+      expect.objectContaining({
+        singleChannelStrategy: expect.objectContaining({
+          variant: expect.objectContaining({
+            channel: 'PUSH',
+            mpushMessageContent: { channel: 'PUSH', messageSubject: 'Hi' },
+          }),
+        }),
+      }),
+    );
+  });
+  it('merges ccsCommDefinition into the data passed to onSave when create succeeds', 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(createCentralCommsMetaId).toHaveBeenCalled());
-    expect(getCentralCommsMetaIds).not.toHaveBeenCalled();
+    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
+    expect(onSave).toHaveBeenCalledWith(
+      expect.objectContaining({
+        ccsCommDefinition: { id: 'cd_123', referenceId: 'ORDER_PLACED_123', version: 1, status: 'DRAFT' },
+      }),
+    );
   });
-  it('uses ouId and module from config.context when provided', async () => {
+  it('preserves a real version of 0 from CCS (not coerced to 1 by a falsy-zero fallback)', async () => {
+    const onSave = jest.fn();
+    createCommDefinition.mockResolvedValueOnce({
+      response: { data: { id: 'cd_123', referenceId: 'ORDER_PLACED_123', status: 'DRAFT', version: { version: 0 } } },
+    });
     renderWithFlow({
       features: {},
-      config: { ...baseConfig, context: { ouId: 42, module: 'LOYALTY' }, features: {} },
+      config: ccsConfig,
       initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
+      onSave,
     });
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
-    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalled());
-    expect(createCentralCommsMetaId).toHaveBeenCalledWith(
+    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
+    expect(onSave).toHaveBeenCalledWith(
       expect.objectContaining({
-        centralCommsPayload: expect.objectContaining({ ouId: 42, module: 'LOYALTY' }),
+        ccsCommDefinition: { id: 'cd_123', referenceId: 'ORDER_PLACED_123', version: 0, status: 'DRAFT' },
       }),
     );
   });
+  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);
+    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');
+  });
+  it('uses config.context.referenceId and description when provided', async () => {
+    renderWithFlow({
+      features: {},
+      config: { ...ccsConfig, context: { ...ccsConfig.context, referenceId: 'ORDER_PLACED', description: 'desc' } },
+      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' }),
+    );
+  });
   it('skips CCS entirely when useCCS is false', async () => {
     const onSave = jest.fn();
     renderWithFlow({
       features: {},
-      config: { ...baseConfig, useCCS: false, features: {} },
+      config: { ...ccsConfig, useCCS: false },
+      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();
+    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,
     });
@@ -442,14 +559,34 @@ describe('handleSave — CCS flow', () => {
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
     await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(createCentralCommsMetaId).not.toHaveBeenCalled();
+    expect(createCommDefinition).not.toHaveBeenCalled();
   });
-  it('still calls onSave when createCentralCommsMetaId rejects', async () => {
-    createCentralCommsMetaId.mockRejectedValue(new Error('Network error'));
+  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();
+  });
+  it('still calls onSave (without ccsCommDefinition) when createCommDefinition rejects', async () => {
+    createCommDefinition.mockRejectedValue(new Error('Network error'));
+    const onSave = jest.fn();
+    renderWithFlow({
+      features: {},
+      config: ccsConfig,
       initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
       onSave,
     });
@@ -457,12 +594,35 @@ 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',
+    });
+    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 createCentralCommsMetaId when contentItems is empty', async () => {
+  it('skips createCommDefinition when contentItems is empty', async () => {
     const onSave = jest.fn();
     renderWithFlow({
       features: {},
+      config: ccsConfig,
       initialData: { contentItems: [] },
       onSave,
     });
@@ -470,12 +630,13 @@ describe('handleSave — CCS flow', () => {
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
     await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
-    expect(createCentralCommsMetaId).not.toHaveBeenCalled();
+    expect(createCommDefinition).not.toHaveBeenCalled();
   });
-  it('includes additionalSettings derived from dynamicControls in the payload', async () => {
+  it('places additionalSettings derived from dynamicControls under settings.additionalSettings (not delivery settings)', async () => {
     renderWithFlow({
       features: {},
+      config: ccsConfig,
       initialData: {
         contentItems: [{ channel: 'SMS', templateData: {} }],
         dynamicControls: {
@@ -489,14 +650,36 @@ describe('handleSave — CCS flow', () => {
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
-    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalled());
-    const payload = createCentralCommsMetaId.mock.calls[0][0];
-    expect(payload.centralCommsPayload.smsDeliverySettings.additionalSettings).toEqual({
+    await waitFor(() => expect(createCommDefinition).toHaveBeenCalled());
+    const payload = createCommDefinition.mock.calls[0][0];
+    expect(payload.settings.additionalSettings).toEqual({
       useTinyUrl: true,
       encryptUrl: true,
       linkTrackingEnabled: true,
       userSubscriptionDisabled: true,
     });
+    expect(payload.singleChannelStrategy.variant.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.variant.smsDeliverySettings.channelSettings).toEqual({
+      channel: 'SMS',
+      domainId: 1001,
+      gsmSenderId: 'BRAND1',
+    });
   });
 });
@@ -552,39 +735,34 @@ describe('optional chaining safety', () => {
     jest.clearAllMocks();
   });
-  it('defaults ouId to -1 when config.context is absent', async () => {
-    createCentralCommsMetaId.mockResolvedValue({ response: { data: { id: 'x' } } });
+  it('does not crash and skips CCS when config.context is entirely absent', 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(createCentralCommsMetaId).toHaveBeenCalled());
-    expect(createCentralCommsMetaId).toHaveBeenCalledWith(
-      expect.objectContaining({
-        centralCommsPayload: expect.objectContaining({ ouId: -1 }),
-      }),
-    );
+    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
+    expect(createCommDefinition).not.toHaveBeenCalled();
   });
-  it('defaults module to consumer.toUpperCase() when config.context.module absent', async () => {
-    createCentralCommsMetaId.mockResolvedValue({ response: { data: { id: 'x' } } });
+  it('does not crash when config.context is present but empty', async () => {
+    const onSave = jest.fn();
     renderWithFlow({
       features: {},
-      config: { ...baseConfig, consumer: 'loyalty', features: {} },
+      config: { ...baseConfig, context: {}, features: {} },
       initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
+      onSave,
     });
     await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
-    await waitFor(() => expect(createCentralCommsMetaId).toHaveBeenCalled());
-    expect(createCentralCommsMetaId).toHaveBeenCalledWith(
-      expect.objectContaining({
-        centralCommsPayload: expect.objectContaining({ module: 'LOYALTY' }),
-      }),
-    );
+    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
+    expect(createCommDefinition).not.toHaveBeenCalled();
   });
   it('initializes channel from config.channel when initialData.channel is absent', () => {

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

@@ -108,6 +108,22 @@ 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 CHANGED Viewed

@@ -38,6 +38,72 @@ 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 sample payloads, verified against real create responses). Kept
+// separate from CHANNEL_CONTENT_KEY_MAP/CHANNEL_DELIVERY_KEY_MAP below, which
+// are the legacy messageMeta payload's keys.
+export const CCS_CHANNEL_CONTENT_KEY_MAP = {
+  SMS: 'smsMessageContent',
+  EMAIL: 'emailMessageContent',
+  WHATSAPP: 'whatsappMessageContent',
+  // CCS's channel enum value for mobile push is 'PUSH' (verified against real
+  // create responses), even though the content/delivery keys stay 'mpush*'.
+  PUSH: 'mpushMessageContent',
+  INAPP: 'inAppMessageContent',
+  ANDROID: 'androidMessageContent',
+  IOS: 'iosMessageContent',
+  ZALO: 'zaloMessageContent',
+  VIBER: 'viberMessageContent',
+  LINE: 'lineMessageContent',
+  WEBPUSH: 'webPushMessageContent',
+  RCS: 'rcsMessageContent',
+};
+// contentItem.templateData is built by the legacy CreativesContainer content
+// editor (messageMeta-oriented field names) — for channels whose legacy field
+// names don't match CCS's own content schema, transform before sending rather
+// than dumping templateData as-is. EMAIL: legacy stores the subject as
+// `emailSubject` (see getContentBody.js), CCS expects `messageSubject`;
+// `messageBody` already matches CCS's name by coincidence. Channels absent
+// here are passed through unchanged (their legacy/CCS field names already
+// align, e.g. SMS's `message`/`messageBody`).
+export const CCS_CONTENT_TRANSFORMS = {
+  EMAIL: (templateData = {}) => ({
+    messageSubject: templateData.emailSubject,
+    messageBody: templateData.messageBody,
+  }),
+};
+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',
+};
+// The UI's own internal channel identifier for mobile push is 'MOBILEPUSH'
+// (see CreativesContainer/constants.js MOBILE_PUSH), but CCS's channel enum
+// value is 'PUSH' (verified against real create responses — the content/
+// delivery KEYS still say 'mpush*', only the enum VALUE is 'PUSH') —
+// normalize before building the CCS payload. Every other channel identifier
+// matches CCS as-is.
+export const CCS_CHANNEL_NAME_MAP = {
+  MOBILEPUSH: 'PUSH',
+};
 // 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 CHANGED Viewed

@@ -9,7 +9,6 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import CommunicationFlow from './CommunicationFlow';
-import { hasSupportEngagementModule } from '../../utils/common';
 const CommunicationFlowContainer = ({
   config,
@@ -18,22 +17,16 @@ const CommunicationFlowContainer = ({
   onCancel,
   onChange,
   ...otherProps
-}) => {
-  if (!hasSupportEngagementModule()) {
-    return null;
-  }
-  return (
-    <CommunicationFlow
-      config={config}
-      initialData={initialData}
-      onSave={onSave}
-      onCancel={onCancel}
-      onChange={onChange}
-      {...otherProps}
-    />
-  );
-};
+}) => (
+  <CommunicationFlow
+    config={config}
+    initialData={initialData}
+    onSave={onSave}
+    onCancel={onCancel}
+    onChange={onChange}
+    {...otherProps}
+  />
+);
 CommunicationFlowContainer.propTypes = {
   config: PropTypes.shape({
@@ -87,11 +80,13 @@ CommunicationFlowContainer.propTypes = {
         controls: PropTypes.array,
       }),
     }),
-    context: PropTypes.object, // ouId, campaignId, programId, etc.
-    useCCS: PropTypes.bool, // If false, skips CCS bulk-claim-approve on save. Defaults to true.
+    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.
   }).isRequired,
   initialData: PropTypes.object, // for edit/preview mode
-  onSave: PropTypes.func.isRequired, // (data) => void - called when user saves
+  onSave: PropTypes.func.isRequired, // (data) => void - called when user saves; data.ccsCommDefinition
+  // ({id, referenceId, version, status}) is present when the CCS create succeeded
   onCancel: PropTypes.func.isRequired, // () => void - called when user cancels
   onChange: PropTypes.func, // (data) => void - optional, called on data changes
 };

package/v2Containers/CommunicationFlow/messages.js CHANGED Viewed

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

package/v2Containers/CreativesContainer/index.js CHANGED Viewed

@@ -1441,7 +1441,6 @@ export class Creatives extends React.Component {
             cardType = "",
             cardSettings = {},
             accountId = "",
-            vfTemplateId,
           } = get(versions, 'base.content.RCS.rcsContent', {});
           const firstCardFromSubmit = Array.isArray(cardContentFromSubmit)
             ? cardContentFromSubmit[0]
@@ -1470,7 +1469,6 @@ export class Creatives extends React.Component {
             creativeName: name,
             rcsContent,
             accountId,
-            ...(vfTemplateId && { templateId: vfTemplateId }),
             ...(includeRootRcsCardVarMappedOnConsumerPayload
               ? { rcsCardVarMapped: cardVarMappedFromFirstRcsCard }
               : {}),

package/v2Containers/Rcs/constants.js CHANGED Viewed

@@ -32,7 +32,6 @@ export const RCS_STATUSES = {
 export const HOST_ICS="rcsicsbulk";
 export const HOST_INFOBIP="rcsinfobipbulk";
-export const HOST_VALUE_FIRST="rcsvaluefirstbulk";
 export const contentType = {
   text_message: 'text_message',

package/v2Containers/Rcs/index.js CHANGED Viewed

@@ -358,8 +358,6 @@ export const Rcs = (props) => {
   const [accessToken, setAccessToken] = useState('');
   const [hostName, setHostName] = useState('');
   const [accountName, setAccountName] = useState('');
-  const [botId, setBotId] = useState('');
-  const [vfTemplateId, setVfTemplateId] = useState('');
   const isHostInfoBip = hostName === HOST_INFOBIP;
   const isHostIcs = hostName === HOST_ICS;
@@ -383,14 +381,12 @@ export const Rcs = (props) => {
       setHostName(accountObj.hostName || '');
       setAccountName(accountObj.name || '');
       setRcsAccount(accountObj.id || '');
-      setBotId(accountObj.connectionProperties?.agentid || '');
     } else {
       setAccountId('');
       setWecrmAccountId('');
       setAccessToken('');
       setHostName('');
       setAccountName('');
-      setBotId('');
     }
   }, [accountData.selectedRcsAccount]);
@@ -1391,8 +1387,6 @@ export const Rcs = (props) => {
         const rcsAccountId = get(details, 'versions.base.content.RCS.rcsContent.accountId', '');
         setRcsAccount(rcsAccountId);
       }
-      const storedVfTemplateId = get(details, 'versions.base.content.RCS.rcsContent.vfTemplateId', '');
-      if (storedVfTemplateId) setVfTemplateId(storedVfTemplateId);
       applySmsFallbackHydration({
         details,
@@ -2830,8 +2824,7 @@ const onTitleAddVar = () => {
                   }
                 ],
                 contentType: isFullMode ? templateType : RICHCARD,
-                ...(isFullMode && {accountId, accessToken, accountName, hostName, botId}),
-                ...(vfTemplateId && { vfTemplateId }),
+                ...(isFullMode && {accountId:accountId, accessToken: accessToken, accountName: accountName, hostName: hostName}),
               },
               ...smsFallbackPayloadContent,
             },
@@ -2971,7 +2964,6 @@ const onTitleAddVar = () => {
     accountName,
     hostName,
     smsRegister,
-    vfTemplateId,
   ]);
   /**