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

nextjs upgrades & security patch #978

Merged
merged 7 commits into from
Nov 6, 2024

Conversation

ElishaKay
Copy link
Collaborator

@ElishaKay ElishaKay commented Nov 6, 2024

Ready for merge 👍

  • Fixed issue of websocket connection dying midsession when user tries to chat with report
  • Fixed the issue of only the first reportBlock being shown to user - instead, only the final reportBlock is shown to the user - thereby making the reflection & revision process of the multi_agents flow more transparent to the user
  • Fixed the issue of duplicate sourceBlocks shown to user
  • consolidate multiple sourceBlocks and reportBlocks into 1 sourceBlock and 1 reportBlock
  • toggle images in report with left & right buttons on desktop or swiping left & right on mobile
  • Removed the getConfig & setConfig routes, and the "API Variables" tab on the frontend - until we think through how to restrict access to those routes

Notes: keeping websocket connection alive with continuous ping-pong game between frontend and backend

@ElishaKay ElishaKay changed the title keeping websocket connection alive with pings keeping websocket connection alive with pings & security patch Nov 6, 2024
@ElishaKay ElishaKay changed the title keeping websocket connection alive with pings & security patch websocket fix & security patch Nov 6, 2024
@ElishaKay ElishaKay changed the title websocket fix & security patch nextjs upgrades & security patch Nov 6, 2024
@assafelovic assafelovic merged commit 2037d97 into assafelovic:master Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants