Skip to content

Commit dc1dc39

Browse files
authored
[ResponseOps][Connectors] Fix missing action variables in Slack Web API text message mode (elastic#259499)
Resolves elastic#259496 <img width="1159" height="724" alt="Screenshot 2026-03-25 at 9 23 24 AM" src="https://github.com/user-attachments/assets/8eefc9ef-8a76-445a-bdec-d556a4ecd7ef" />
1 parent ab812aa commit dc1dc39

2 files changed

Lines changed: 28 additions & 0 deletions

File tree

x-pack/platform/plugins/shared/stack_connectors/public/connector_types/slack_api/slack_params.test.tsx

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -417,6 +417,33 @@ describe('SlackParamsFields', () => {
417417
});
418418
});
419419

420+
it('renders action variables in Text mode', async () => {
421+
const messageVariables = [
422+
{
423+
name: 'alert.id',
424+
description: 'The alert ID',
425+
},
426+
];
427+
428+
appMockRenderer.render(
429+
<SlackParamsFields
430+
actionParams={{}}
431+
editAction={editAction}
432+
index={0}
433+
errors={{ message: [] }}
434+
actionConnector={actionConnector}
435+
messageVariables={messageVariables}
436+
/>
437+
);
438+
439+
const addVariableButton = screen.getByTestId('webApiTextAddVariableButton');
440+
expect(addVariableButton).toBeInTheDocument();
441+
expect(addVariableButton).not.toBeDisabled();
442+
443+
await userEvent.click(addVariableButton);
444+
expect(await screen.findByTestId('variableMenuButton-alert.id')).toBeInTheDocument();
445+
});
446+
420447
it('cannot set channels not in the allow list', async () => {
421448
appMockRenderer.render(
422449
<SlackParamsFields

x-pack/platform/plugins/shared/stack_connectors/public/connector_types/slack_api/slack_params.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -240,6 +240,7 @@ const SlackParamsFieldsComponent: React.FunctionComponent<ParamsProps> = ({
240240
<TextMessage
241241
index={index}
242242
onChange={onTextChange}
243+
messageVariables={messageVariables}
243244
value={textValue}
244245
errors={(errors.text ?? []) as string[]}
245246
/>

0 commit comments

Comments
 (0)