-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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 message table and update message functionality #2290
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Your Render PR Server URL is https://langflow-pr-2290.onrender.com. Follow its progress at https://dashboard.render.com/web/srv-cpsce4rv2p9s73ba6pn0. |
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.
LGTM
299eaa5
to
4365e3d
Compare
f543763
to
3027f6e
Compare
Deployment failed with the following error:
|
241e6ba
to
22b131b
Compare
Deployment failed with the following error:
|
c8ea85d
to
24033cf
Compare
Deployment failed with the following error:
|
…int for auto_login
…t statement for better performance and readability 📝 (memory.py): Refactor delete_messages function to use SQLAlchemy delete statement for better performance and readability 📝 (monitor/schema.py): Update MessageModel to use UUID type for id and flow_id for consistency and better data handling
…messages in MonitorService
…n langflow module.
…rn Message objects
f97dd02
to
12f35f2
Compare
This pull request includes the following changes:
Added a new table called "message" to the database, allowing for storing and retrieving messages in the application.
Refactored the add_messages function to use a database session for adding messages to the monitor service, improving reliability.