-
Notifications
You must be signed in to change notification settings - Fork 9.8k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat(AI Agent Node): Update descriptions and titles for Chat Trigger options in AI Agents and Memory #12155
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
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.
Left a minor comment but good to go on my end
value: 'fromInput', | ||
description: 'Looks for an input field called sessionId', | ||
description: | ||
'Looks for an input field called sessionId that is coming from a directly connected Chat Trigger', |
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.
Nit: Does it make sense to wrap the sessionId
in quotes or backticks to make it standout a bit more?
n8n Run #8321
Run Properties:
|
Project |
n8n
|
Branch Review |
ADO-2947
|
Run status |
Passed #8321
|
Run duration | 04m 38s |
Commit |
bb88925e8b: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 CharlieKolb 🗃️ e2e/*
|
Committer | Charlie Kolb |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
480
|
View all changes introduced in this branch ↗︎ |
✅ All Cypress E2E specs passed |
✅ All Cypress E2E specs passed |
Got released with |
Summary
Update the messaging for AI Agent and Memory parameters related to using ChatTrigger automatically.
Related Linear tickets, Github issues, and Community forum posts
https://linear.app/n8n/issue/ADO-2947/feature-tweak-the-prompt-source-parameter
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)