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.

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

[FEAT] Edit message button #1392

Merged
merged 8 commits into from
Jun 6, 2024
Merged

[FEAT] Edit message button #1392

merged 8 commits into from
Jun 6, 2024

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
@PaoloC68
Copy link

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT]: Edit chat interaction
3 participants