-
Notifications
You must be signed in to change notification settings - Fork 10
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
Review ChatHttpResponse
#880
Comments
pmarsh-scottlogic
added
refactor
Improve code quality
frontend
Requires work on the frontend
backend
Requires work on the backend
labels
Mar 26, 2024
Here's all the cases: normal chatchat with transformationwin leveldefence triggerdefence alertopenAI errorpaste the following code in the throw new Error(
'429: You are being rate limited. Please try again in 3 minutes.'
); other errorpaste defence alert AND win level
transformation AND win level
transformation AND defence alerttransformation AND defence triggertransformation AND defence alert AND win level
multiple defence triggers
defence trigger and alert
multiple defence alerts |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When we access the API, by
POST /openai/chat
, we return an object that looks like thisAfter #873 we add
wonLevelMessage
as well.What's the problem? Well all of the following properties represent something that might be added to the chat history:
reply
,transformedMessage
,wonLevelMessage
,openAIErrorMessage
,transformedMessageInfo
, or at least displayed on the frontend's chatHistory. We should think about perhaps just returning a list ofChatMessage
sThis will require some investigation (about when, where and how the above messages get added to the font/backend chats), and a bit of poking around on both the front and back end, and then probably lots of test changes.
The text was updated successfully, but these errors were encountered: