Skip to content

Feature/chat on demand#121

Merged
samcraftt merged 3 commits intomainfrom
feature/chat-on-demand
Feb 17, 2026
Merged

Feature/chat on demand#121
samcraftt merged 3 commits intomainfrom
feature/chat-on-demand

Conversation

@zhangdjr
Copy link
Collaborator

Implemented two features actually:

  1. Implemented chat on demand for certain UI components, where users confused about how to use the app can click on a circluar question mark to receive info while being able to ask follow up questions.
  2. Implemented a chat saving feature where users can save past chats sorta like in chatgpt.

@samcraftt
Copy link
Collaborator

Sweet additions.

Since the first chat on demand message is always delivered to the user with no input from them, should we just hardcode the first message, so as to not have to wait for the chatbot to respond if it's essentially going to say the same thing every time? Then on subsequent queries we can generate more info via chatbot to answer follow up questions.

If it's too tedious then nvm, just a thought

@zhangdjr
Copy link
Collaborator Author

Sweet additions.

Since the first chat on demand message is always delivered to the user with no input from them, should we just hardcode the first message, so as to not have to wait for the chatbot to respond if it's essentially going to say the same thing every time? Then on subsequent queries we can generate more info via chatbot to answer follow up questions.

If it's too tedious then nvm, just a thought

I just didn't want to write an entire info blurb for every single ui component, I thought it would be easier to just have the LLM retreive the info needed using RAG from the docs we gave it. And after some time, users will probably be familair enough to stop asking questions about UI. Just my two cents. But if you really want, I can just hard code the info blurbs. What do you guys think? @akgohain @samcraftt

@samcraftt
Copy link
Collaborator

True, if we were to save a default info blurb for every component we provide this for that'd be clutter. So looks good to me!

@samcraftt samcraftt merged commit 4679c0c into main Feb 17, 2026
2 checks passed
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