Skip to content

Conversation

@RAWx18
Copy link

@RAWx18 RAWx18 commented Jul 4, 2025

When users accessed the chatbot page for the first time, an error was thrown:
"Knowledge base not initialized"

This occurred because no data was available in the knowledge base on initial load, causing the system to fail when trying to render or access undefined content.

Screenshot (Before):
Screenshot From 2025-07-04 14-44-44

Screenshot (After):
Screenshot From 2025-07-04 15-08-51

Signed-off-by: RAWx18 <rawx18.dev@gmail.com>
@sancheet230
Copy link

sancheet230 commented Jul 4, 2025

@RAWx18 I think it would be better if we add demo data
that would be more convenient

@gcapuzzi
Copy link
Contributor

gcapuzzi commented Jul 4, 2025

Or we will share the admin user guide and they will add demo data from their documents

@RAWx18
Copy link
Author

RAWx18 commented Jul 4, 2025

Or we will share the admin user guide and they will add demo data from their documents

Yes i made it this way so that they can try on their data. Because proving on our data, doesn't prove for our clients that it is good, we need to work on their data.

@RAWx18
Copy link
Author

RAWx18 commented Jul 4, 2025

So i allowed them to enter their data and test it

@sancheet230
Copy link

@gcapuzzi Can't we just land then on config page at first and then give option to build knowledge base there only and then redirect to chatbot page after building knowledge base

@gcapuzzi
Copy link
Contributor

gcapuzzi commented Jul 4, 2025

The client explicitly asked me to start with the chatbot page

@sancheet230
Copy link

@gcapuzzi I am suggesting this for admin not users since admin is the one to feed data so we can do that for admins and start chatbot directly for users

@gcapuzzi
Copy link
Contributor

gcapuzzi commented Jul 4, 2025

@sancheet230 we could maintain the same behavior for users and admin..at the same time, providing "how to add documents" is a plus

@RAWx18
Copy link
Author

RAWx18 commented Jul 6, 2025

So do i need to change it?

@gcapuzzi gcapuzzi merged commit 2f2494b into hyperledger-labs:agents Jul 6, 2025
5 checks passed
@gcapuzzi
Copy link
Contributor

gcapuzzi commented Jul 6, 2025

Merged: I'm testing

@gcapuzzi
Copy link
Contributor

gcapuzzi commented Jul 8, 2025

@RAWx18 It seems to work file, so no changes nedeed!

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.

3 participants