-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Add Slack Trigger node #2057
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Slack Trigger node #2057
Conversation
✅ Deploy Preview for n8n-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Overall readability score: 45.26 (🟢 +0.01)
View detailed metrics🟢 - Shows an increase in readability
Averages:
View metric targets
|
docs/integrations/builtin/trigger-nodes/n8n-nodes-base.slacktrigger.md
Outdated
Show resolved
Hide resolved
docs/integrations/builtin/trigger-nodes/n8n-nodes-base.slacktrigger.md
Outdated
Show resolved
Hide resolved
docs/integrations/builtin/trigger-nodes/n8n-nodes-base.slacktrigger.md
Outdated
Show resolved
Hide resolved
docs/integrations/builtin/trigger-nodes/n8n-nodes-base.slacktrigger.md
Outdated
Show resolved
Hide resolved
|
||
## Slack configuration | ||
|
||
To use this node you first need to create an application in Slack and enable event subscriptions. For the **Request URL** input the production or test URL displayed in the webhooks section of the node. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also mention that they should install the App in a Slack workspace and that it should be channel member for some events? Maybe it's obvious but I forgot while testing 🙃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is a good idea I will add that shortly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Joffcom do you still need to do this? Or is this PR ready to merge?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New plan: I want to link this from the release note. So merging - can always add more detail later if you still need to.
n8n-io/n8n#9190