-
Notifications
You must be signed in to change notification settings - Fork 38
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
Enhance/chat #975
Enhance/chat #975
Conversation
❌ Tests failed for this pull request. 😞 |
Deploying packrat with Cloudflare Pages
|
iOS app build failed! |
05c1af1
to
a3f9ab4
Compare
Hi @pinocchio-life-like , Thanks for the update! Everything looks great. However, I'd like to suggest one addition. When I tried minimizing the chat (by clicking the chat icon only), I noticed that the text I entered disappeared. Could we update this so that the chat history is retained? Perhaps we could have the "X" button on top clear the chat history instead. |
Hi @EngrAJ8888 which text do you mean exactly? the one on the input field or those you have already sent? if its the one you have already sent, since the backend is returning error, sent messages will not be retained. but if you mean the text field being cleared, I will address that. let me know! |
Hi @pinocchio-life-like , I was referring to the text I inputted into the chat AI. I've also attached a photo for reference. Does this mean we'll leave it as is for now, considering it's a backend issue? Thanks! |
@EngrAJ8888 Yes, I believe. I have left message for on discord. check that. |
Hi @pinocchio-life-like - Please refer to the video for clarification on the issue I raised. The text should remain if I only minimize the chat box. However, if the problem lies in the backend, we can go ahead and close this out. Thank you! 15.mp4 |
@EngrAJ8888 Yes I am sure that this is OpenAI API key not being set for the backend. If this suffices the issue you raised you may close this. Its also okay if you dont since I am working on other enhancements regarding chat. |
Update, it works now. |
Hi @pinocchio-life-like - thanks for the update! It's looking great! @andrew-bierman - this is good for merging. |
No description provided.