Skip to content
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: Add Redis Chat Memory #3832

Merged

Conversation

MarceloNunesAlves
Copy link
Contributor

This PR is about including the Redis component as a cache for chat memory.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Sep 17, 2024
@github-actions github-actions bot added the enhancement New feature or request label Sep 17, 2024
@dosubot dosubot bot added the python Pull requests that update Python code label Sep 17, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-3832.dmtpw4p5recq1.amplifyapp.com

Copy link
Contributor

@ogabrielluiz ogabrielluiz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @MarceloNunesAlves !
Sorry for the slow response.
LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 4, 2024
@ogabrielluiz ogabrielluiz enabled auto-merge (squash) October 4, 2024 16:55
@ogabrielluiz ogabrielluiz merged commit b7aed91 into langflow-ai:main Oct 4, 2024
28 checks passed
ogabrielluiz added a commit that referenced this pull request Oct 4, 2024
* feat: Add Redis Chat Memory

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* Refactor password handling in RedisIndexChatMemory to use a local variable

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Gabriel Luiz Freitas Almeida <[email protected]>
@MarceloNunesAlves
Copy link
Contributor Author

Thanks @MarceloNunesAlves ! Sorry for the slow response. LGTM

No problem! Thank you so much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lgtm This PR has been approved by a maintainer python Pull requests that update Python code size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants