-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Initial commit for respond-to-chat #3339
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
Conversation
pdwarf
commented
Jul 9, 2025
- Added basic scaffolding for the docs page
- Sorted this to be within the Chat Trigger docs as the node itself is also part of the Chat Trigger code directory and requires a Chat Trigger to work, see PR: https://github.com/n8n-io/n8n/pull/12546/files#diff-d9b8bee18b0196641e173658f644a5069cb9e01a1ebb315151bd5a85bf13add1
✅ Deploy Preview for n8n-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Hey @pdwarf, thanks for the PR! It looks pretty good so far. I've left a few comments that might help. Additionally:
- I spoke with David R. briefly about whether we should nest this within the Chat Trigger node or not. He had a really good point by asking how we do things with the Form node and Form Trigger node since those have the same type of relationship you describe here (The Form node depends on the Form Trigger node being present). Since we don't nest those, we should probably follow that pattern here and break this out into its own node page. I can do that once we're happy with the rest of the content if you agree that sounds reasonable.
- I'll also need to add the page to the
nav.yml
file, so that it shows up in the site navigation. This is easy to do as I make the other changes, but for future reference, that's where you'd go to add the page to the site nav. - Do we have a current test instance I can look at? The one in the initial ticket has since been deleted. Probably best to share in Linear or Slack.
Let me know if you have any other questions about the suggestions or comments. Thanks again!
docs/integrations/builtin/core-nodes/n8n-nodes-langchain.chattrigger/respondtochat.md
Outdated
Show resolved
Hide resolved
docs/integrations/builtin/core-nodes/n8n-nodes-langchain.chattrigger/respondtochat.md
Outdated
Show resolved
Hide resolved
docs/integrations/builtin/core-nodes/n8n-nodes-langchain.chattrigger/respondtochat.md
Outdated
Show resolved
Hide resolved
docs/integrations/builtin/core-nodes/n8n-nodes-langchain.chattrigger/respondtochat.md
Outdated
Show resolved
Hide resolved
docs/integrations/builtin/core-nodes/n8n-nodes-langchain.chattrigger/respondtochat.md
Outdated
Show resolved
Hide resolved
docs/integrations/builtin/core-nodes/n8n-nodes-langchain.chattrigger/respondtochat.md
Outdated
Show resolved
Hide resolved
…rigger/respondtochat.md Co-authored-by: Justin Ellingwood <justin@n8n.io>
…rigger/respondtochat.md Co-authored-by: Justin Ellingwood <justin@n8n.io>
…rigger/respondtochat.md Co-authored-by: Justin Ellingwood <justin@n8n.io>
…rigger/respondtochat.md Co-authored-by: Justin Ellingwood <justin@n8n.io>
…rigger/respondtochat.md Co-authored-by: Justin Ellingwood <justin@n8n.io>
…rigger/respondtochat.md Co-authored-by: Justin Ellingwood <justin@n8n.io>
Yeah that makes sense, happy to follow this convention for splitting it out and not nesting it! I've accepted all the suggestions and will add a bit more information as we've had to tweak the "Response Mode" of the Chat Trigger in lieu of the Respond to Chat node. |
Summary
|
Thanks @pdwarf! We'll hold off on merging this until the PR is ready to go in the main repo. |
Holding off on merging until this is released. |