Add explicit button for session summary #5750
Conversation
|
Your Render PR Server URL is https://frontend-pr-5750.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-cidfcmdgkuvnbpa2m9p0. |
Vadman97
left a comment
There was a problem hiding this comment.
unrelated to this PR, but just noticed that insights can come out of order. can we update the prompt to fix that?
SpennyNDaJets
left a comment
There was a problem hiding this comment.
Wanted to just raise the following things I found from click testing:
- The Button is not there after generating a response, and then changing sessions. Not sure if its automatically hitting our endpoint, or stuck in a loading loop though 🤔
- The times seem to be off, but not sure if this is a local/unrelated issue
|
Vadman97
left a comment
There was a problem hiding this comment.
small note on the loading button
also the insights seem to be broken in the lambda
https://www.loom.com/share/14373574d53942e390988be0efd2fffa
Vadman97
left a comment
There was a problem hiding this comment.
realized that the lambda might be broken in the render.com preview since that doesn't deploy the backend changes, in which case disregard!
Summary
Added button to trigger session insights as specified in #5687
Also fixed the backend insights retry function when local server is unavailable.