opensource/n8n
3
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2026-10-30 19:35:36 +08:00
Code Issues Packages Projects Releases Wiki Activity

feat(Slack Node): Add app_home_opened as a dedicated trigger event (#28626)

Browse Source 
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Roman Davydchuk <roman.davydchuk@n8n.io>
This commit is contained in:
Jon
2026-04-17 19:13:53 +00:00
committed by GitHub
co-authored by Claude Sonnet 4.6 Roman Davydchuk
parent ff950e5840
commit f1dab3e295
3 changed files with 46 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/nodes-base/nodes/Slack/SlackTrigger.node.ts
+7 -1
View File
@@ -68,6 +68,11 @@ export class SlackTrigger implements INodeType {
value: 'any_event',
description: 'Triggers on any event',
},
{
name: 'App Home Opened',
value: 'app_home_opened',
description: "When a user opens your app's Home tab",
},
{
name: 'Bot / App Mention',
value: 'app_mention',
@@ -352,7 +357,8 @@ export class SlackTrigger implements INodeType {
return {};
}
if (eventType !== 'team_join') {
const eventsWithoutChannel = ['team_join', 'app_home_opened'];
if (!eventsWithoutChannel.includes(eventType)) {
eventChannel =
req.body.event.channel ?? req.body.event.item?.channel ?? req.body.event.channel_id;
packages/nodes-base/nodes/Slack/test/SlackTrigger.test.ts
+37
View File
@@ -506,6 +506,43 @@ describe('SlackTrigger Node', () => {
expect(out.user_resolved).toBe(mockRequest.body.event.user.name);
});
it('should trigger on app_home_opened event without channel validation', async () => {
mockWebhookFunctions.getNodeParameter.mockImplementation(
(paramName: string, defaultValue?: any) => {
switch (paramName) {
case 'trigger':
return ['app_home_opened'];
case 'watchWorkspace':
return false;
case 'options':
return {};
default:
return defaultValue;
}
},
);
const mockRequest = {
body: {
type: 'event_callback',
event: {
type: 'app_home_opened',
user: 'U061F7AUR',
channel: 'D0LAN2Q65',
tab: 'home',
event_ts: '1515449522000016',
},
},
};
mockWebhookFunctions.getRequestObject.mockReturnValue(mockRequest as any);
const result = await slackTrigger.webhook!.call(mockWebhookFunctions);
expect(result.workflowData).toBeDefined();
expect(result.workflowData![0][0].json).toEqual(mockRequest.body.event);
});
it('should handle url_verification challenge', async () => {
const mockRequest = {
body: {
packages/nodes-base/utils/sendAndWait/test/util.test.ts
+2 -2
View File
@@ -279,7 +279,7 @@ describe('Send and Wait utils tests', () => {
expect(mockSetHeader).toHaveBeenCalledWith(
'Content-Security-Policy',
'sandbox allow-downloads allow-forms allow-modals allow-orientation-lock allow-pointer-lock allow-popups allow-presentation allow-scripts allow-top-navigation-by-user-activation allow-top-navigation-to-custom-protocols',
'sandbox allow-downloads allow-forms allow-modals allow-orientation-lock allow-pointer-lock allow-popups allow-popups-to-escape-sandbox allow-presentation allow-scripts allow-top-navigation-by-user-activation allow-top-navigation-to-custom-protocols',
);
expect(mockRender).toHaveBeenCalledWith('form-trigger', {
@@ -365,7 +365,7 @@ describe('Send and Wait utils tests', () => {
expect(mockSetHeader).toHaveBeenCalledWith(
'Content-Security-Policy',
'sandbox allow-downloads allow-forms allow-modals allow-orientation-lock allow-pointer-lock allow-popups allow-presentation allow-scripts allow-top-navigation-by-user-activation allow-top-navigation-to-custom-protocols',
'sandbox allow-downloads allow-forms allow-modals allow-orientation-lock allow-pointer-lock allow-popups allow-popups-to-escape-sandbox allow-presentation allow-scripts allow-top-navigation-by-user-activation allow-top-navigation-to-custom-protocols',
);
expect(mockRender).toHaveBeenCalledWith('form-trigger', {