-
Notifications
You must be signed in to change notification settings - Fork 2.4k
feat: Implement conversation history #524
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
base: main
Are you sure you want to change the base?
Conversation
…and research replays
…log for conversation retrieval
… update return types in TavilySearchWithImages; add console logging for search results
…r-flow into feat/conversations
…ust related tests
…arch_results_with_images.py
…sage count indicators
|
Need it, please review. @WillemJiang |
|
@changxubo, could you take a look at the lint error of the web front-end? |
|
@WillemJiang Fixed ,please review! |
|
@WillemJiang Is this PR ready to be merged? |
|
@WillemJiang @changxubo 你好,请问这个历史回放的代码什么时候可以合并到主分支,谢谢 |
|
或许这个功能在添加用户管理之后再合并到项目中会比较好,否则多用户情况下大家的历史记录不好管理,每个人都能看到别人的历史记录也不太好。 |
的确,我们需要先实现用户管理功能之后才能合并这样的功能。 |
建议可以不用考虑用户管理功能,这个系统比较适合内嵌到已有系统(本身已有用户管理),只要上层做好用户与聊天记录id的对应关系即可。 |
Changes: