θΏ™ζ˜―indexlocζδΎ›ηš„ζœεŠ‘οΌŒδΈθ¦θΎ“ε…₯任何密码
Skip to content

Conversation

@shatfield4
Copy link
Collaborator

@shatfield4 shatfield4 commented May 14, 2024

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

Relevant Issues

resolves #1357

What is in this change?

  • Allow user to edit previously sent messages and regenerate the response using the new message
  • UI implemented
  • Backend functions complete, WIP connecting to frontend
  • Add default workspace icon

Additional Information

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

@shatfield4 shatfield4 self-assigned this May 14, 2024
@shatfield4
Copy link
Collaborator Author

shatfield4 commented May 14, 2024

Bugs:

  • Currently edits correct message and deletes all messages below it if we refresh page after sending messages (the history does not contain the chatId when we send a chat unless we get it from the backend)
  • Have to click the edit button twice for it to hide (something with the useMemo not rerendering the component?)

@timothycarambat timothycarambat removed their assignment May 17, 2024
Refactor of editMessage implementation
TODO: Fix default user profile image
Add User and Assistant workspace response
@timothycarambat timothycarambat marked this pull request as ready for review May 23, 2024 23:34
@timothycarambat timothycarambat added the PR:needs review Needs review by core team label May 23, 2024
@PaoloC68
Copy link
Contributor

PaoloC68 commented Jun 6, 2024

@timothycarambat I really miss this feature, there is any hope to see this soon? Can I help by testing the branch?

@timothycarambat
Copy link
Member

@PaoloC68 I have been in China for the last two weeks so we were taking is slow on code commits. It will be merged soon.

@timothycarambat timothycarambat merged commit 26c2205 into master Jun 6, 2024
@timothycarambat timothycarambat deleted the edit-message-button branch June 6, 2024 19:56
CrackerCat pushed a commit to CrackerCat/anything-llm that referenced this pull request Jul 31, 2024
* WIP edit message feature

* WIP edit message

* WIP editing messages feature

* Fix PFPs
TODO: Fix default user profile image
Add User and Assistant workspace response

* unset PFP changes for later PR

---------

Co-authored-by: timothycarambat <rambat1010@gmail.com>
CrackerCat pushed a commit to CrackerCat/anything-llm that referenced this pull request Aug 1, 2024
* WIP edit message feature

* WIP edit message

* WIP editing messages feature

* Fix PFPs
TODO: Fix default user profile image
Add User and Assistant workspace response

* unset PFP changes for later PR

---------

Co-authored-by: timothycarambat <rambat1010@gmail.com>
CrackerCat pushed a commit to CrackerCat/anything-llm that referenced this pull request Aug 2, 2024
* WIP edit message feature

* WIP edit message

* WIP editing messages feature

* Fix PFPs
TODO: Fix default user profile image
Add User and Assistant workspace response

* unset PFP changes for later PR

---------

Co-authored-by: timothycarambat <rambat1010@gmail.com>
CrackerCat pushed a commit to CrackerCat/anything-llm that referenced this pull request Aug 3, 2024
* WIP edit message feature

* WIP edit message

* WIP editing messages feature

* Fix PFPs
TODO: Fix default user profile image
Add User and Assistant workspace response

* unset PFP changes for later PR

---------

Co-authored-by: timothycarambat <rambat1010@gmail.com>
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
* WIP edit message feature

* WIP edit message

* WIP editing messages feature

* Fix PFPs
TODO: Fix default user profile image
Add User and Assistant workspace response

* unset PFP changes for later PR

---------

Co-authored-by: timothycarambat <rambat1010@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR:needs review Needs review by core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT]: Edit chat interaction

4 participants