Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support multiple chat sessions within the web UI #638

Merged
merged 6 commits into from Feb 11, 2024
Merged

Support multiple chat sessions within the web UI #638

merged 6 commits into from Feb 11, 2024

Conversation

sabaimran
Copy link
Collaborator

@sabaimran sabaimran commented Feb 6, 2024 •
edited by debanjum

Incoming

  • Add support for multiple chat sessions from within the web UI.
  • Allow users to create multiple chat sessions and manage them
  • Give chat session slugs based on the most recent message
  • Update web UI to have a collapsible menu with active chats
  • Move chat routes into a separate file
  • Add some cool UI components for editing the title, deleting conversations from within the web UI
  • Support modification of the conversation title
  • Update UI to add a threedotmenu to each conversation
Chat Sessions UI
Screenshot 2024-02-06 at 17 42 01

Notes:

  • We still need to add this functionality to the desktop app. I'll cover that in a follow-up PR.

Refer #275

@sabaimran
Enable support for multiple chat sessions within the web client
14f00da 
- Allow users to create multiple chat sessions and manage them
- Give chat session slugs based on the most recent message
- Update web UI to have a collapsible menu with active chats
- Move chat routes into a separate file
@sabaimran
Make the collapsible side panel more graceful, improve some styling e…
123d521 
…lements of the new layout
@sabaimran
Support modification of the conversation title
632b006 
- Add a new field to the conversation object
- Update UI to add a threedotmenu to each conversation
@sabaimran
Get the default conversation if a matchine one is not found by id
37966af
github-advanced-security[bot]
github-advanced-security bot found potential problems Feb 6, 2024
View reviewed changes
src/khoj/interface/web/chat.html
fetch(editURL , { method: "PATCH" })
.then(response => response.ok ? response.json() : Promise.reject(response))
.then(data => {
conversationButton.innerHTML = newTitle;

Check warning

Code scanning / CodeQL

DOM text reinterpreted as HTML Medium

DOM text
is reinterpreted as HTML without escaping meta-characters.
@sabaimran sabaimran mentioned this pull request Feb 6, 2024
Merged
debanjum
debanjum approved these changes Feb 11, 2024
View reviewed changes
Copy link
Member

@debanjum debanjum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking forward to playing with these changes. It'll be a relief to be able to separate, organize chat sessions based on areas/topic

src/khoj/routers/api_chat.py Outdated Show resolved Hide resolved
src/khoj/interface/web/utils.html Show resolved Hide resolved
src/khoj/database/adapters/__init__.py Outdated Show resolved Hide resolved
src/khoj/interface/web/assets/khoj.css Show resolved Hide resolved
src/khoj/database/migrations/0031_conversation_title.py Outdated Show resolved Hide resolved
@sabaimran sabaimran merged commit 1412ed6 into master Feb 11, 2024
9 checks passed
@sabaimran sabaimran deleted the features/multiple-chat-sessions branch February 11, 2024 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@debanjum debanjum debanjum approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@sabaimran @debanjum