-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
feat(Postgres Chat Memory Node): Implement Postgres Chat Memory node #10071
Conversation
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.
Looks good! Minor comments, non-blocking.
4 flaky tests on run #5951 ↗︎
Details:
5-ndv.cy.ts • 2 flaky tests
10-undo-redo.cy.ts • 1 flaky test
24-ndv-paired-item.cy.ts • 1 flaky test
Review all test suite changes for PR #10071 ↗︎ |
✅ All Cypress E2E specs passed |
…lify isChatInstance type
cbfa9ad
to
685ddd4
Compare
✅ All Cypress E2E specs passed |
Got released with |
I'm afraid I can't find the Postgres Chat Memory node in n8n@1.51.0. |
@mprytoluk I have just taken a look and it is there for me, How are you trying to find it? |
@Joffcom I believe there was a mistake on my side, because the node is indeed there in n8n@1.51.0. Sorry for the confusion. |
@mprytoluk This type of node will only appear in search results if there's a node that can use it, such as the AI Agent node. You can find it manually by going to "Advanced AI > Other AI Nodes > Memory". |
Summary
BaseChatMessageHistory
&BaseChatMemory
in logWrapper to make sure sub-node input/ouput is correctly populatedDocs: n8n-io/n8n-docs#2241
How to test
Related Linear tickets, Github issues, and Community forum posts
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)