Skip to content

Conversation

@thlehmann-ionos
Copy link

  • Remove chat share/clone endpoints
  • Remove UI components that are not used currently

* The functionality is not used
* Remove unused code

== Notes

* ToolServersModal: imported API functions, but did not use them
* ChatMenu: exported the handlers, but did not call them anymore
  (removed in the past already)

Refs: PRODAI-426
* The functionality is not used
* Frontend code removed in previous commit
* Remove unused code

Refs: PRODAI-426
Related business logic was removed in previous commit.

Refs: PRODAI-426
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes chat sharing functionality from the application, eliminating both backend endpoints and frontend UI components that are no longer needed.

  • Removes share and clone shared chat API endpoints from the backend
  • Removes ShareChatModal component and related UI elements from the frontend
  • Cleans up unused imports and handler functions related to chat sharing

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/routes/s/[id]/+page.svelte Completely removes shared chat view page
src/lib/components/layout/Sidebar/ChatMenu.svelte Removes share and clone chat handler exports
src/lib/components/layout/Sidebar/ChatItem.svelte Removes ShareChatModal import and clone chat functionality
src/lib/components/layout/Navbar.svelte Removes ShareChatModal import and replaces handler with empty function
src/lib/components/chat/ToolServersModal.svelte Removes unused import for share chat APIs
src/lib/components/chat/ShareChatModal.svelte Completely removes the share chat modal component
src/lib/components/chat/Navbar.svelte Removes ShareChatModal import and replaces handler with empty function
src/lib/apis/chats/index.ts Removes share-related API functions
backend/open_webui/routers/chats.py Removes share-related endpoints
backend/open_webui/models/chats.py Removes share-related database methods and updates delete methods
Comments suppressed due to low confidence (2)

backend/open_webui/models/chats.py:1

  • The code attempts to call delete_shared_chat_by_chat_id method which has been removed in this PR, causing a NameError.
import logging

backend/open_webui/models/chats.py:1

  • The code attempts to call delete_shared_chats_by_user_id method which has been removed in this PR, causing a NameError.
import logging

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@thlehmann-ionos thlehmann-ionos merged commit ec2a92f into ionos-dev Oct 14, 2025
1 check 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.

3 participants