Skip to content

Feature/edit delete messages#31

Open
Redesu wants to merge 18 commits intoTiagoMarinho:mainfrom
Redesu:feature/edit-delete-messages
Open

Feature/edit delete messages#31
Redesu wants to merge 18 commits intoTiagoMarinho:mainfrom
Redesu:feature/edit-delete-messages

Conversation

@Redesu
Copy link
Copy Markdown
Contributor

@Redesu Redesu commented Nov 15, 2025

  • Add core functionality for editing and deleting messages
  • Add buttons to the UI that allow the user to edit and delete messages
  • Users can edit and delete messages.
  • Edited/deleted data persists on the backend.
  • Editing a user message should trigger an llm service call.
  • Editing an assistant message should not trigger an llm service call.

Adds 2 buttons for both assistant and user messages, user can delete and edit the messages. It persists on the backend.

Delete
brave_70dcUDp0oC

Edit

If its a assistant message, it is not triggering the llm service
brave_tnnHtMu2af

If its a user message, it triggers the llm service and puts the message bellow the existing message
brave_EmUN4huaUs

Warning

Right now it is keeping both old and new messages from the LLM. Changes can be made if decided to delete the old one

Since both send and edit messages uses stream, a new function was made for a cleaner code.

@Redesu
Copy link
Copy Markdown
Contributor Author

Redesu commented Nov 15, 2025

Resolves Add message editing and deletion #10

@TiagoMarinho
Copy link
Copy Markdown
Owner

This is now blocked by #35

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