jan icon indicating copy to clipboard operation
jan copied to clipboard

feat: Users should be able to edit messages

Open hantran-co opened this issue 1 year ago • 1 comments

Epic: Threads Improvement #1507


Specs & Design

Figma link: https://www.figma.com/file/ytn1nRZ17FUmJHTlhmZB9f/Jan-App?type=design&node-id=3624-167353&mode=design&t=3XalkA6NVYOqVenZ-4

1. Users should be able to see the "Edit" option when they hover on the messages they have sent.

Image

2. Click edit -> be able to edit the message -> Can submit the revised version or cancel

Image

3. Once submit, latest messages below the edited message will be gone and the assistant will re-generate the answer based on the edited version of the message

Image

hantran-co avatar Jan 24 '24 11:01 hantran-co

It would be nice if we could also edit the response of the assistant. It's a very powerful tool: if the assistant got a few facts wrong, it's easier and more efficient to correct manually the response. Then continue the conversation like normal. It's something I do a lot with llama.cpp server in "Completion" mode, and smaller models really need it to be useful.

Hugi-R avatar Feb 09 '24 18:02 Hugi-R

hi @namchuai

  1. Please help correct this issue where trying to edit another message, is only closing the currently editing msg. Users will have to click edit the 2nd time to continue editing :x: Image

  2. On the other hand, when editing a msg without submission, switching to a new thread and going back, the currently editing message is reverted back to before the edit, so any changes made by users in the edit box will be gone❓is this expected, @imtuyethan

Van-QA avatar Feb 18 '24 04:02 Van-QA

  • I edit message A -> don't choose "Submit"
  • Click on edit message B -> edit message A should be automatically closes
  1. This is expected

hantran-co avatar Feb 21 '24 03:02 hantran-co

for No. 1 currently users will have to click twice, when moving from one editing box to another.

  • 1st click only close the current editing
  • 2nd click will open new editing => need correction as well

No. 2 at least when users moved to another thread, and go back, if the editing text not saved, the edit box should be closed

Van-QA avatar Feb 21 '24 04:02 Van-QA

Tested and looking good on Jan v0.4.6-280 ✅

Van-QA avatar Feb 22 '24 04:02 Van-QA

It would be nice if we could also edit the response of the assistant. It's a very powerful tool: if the assistant got a few facts wrong, it's easier and more efficient to correct manually the response. Then continue the conversation like normal. It's something I do a lot with llama.cpp server in "Completion" mode, and smaller models really need it to be useful.

I wonder if there's another issue tracking this request or if this isn't currently being considered? Checked the latest nightly and I see we can only edit the user's input.

user-01010 avatar Feb 24 '24 17:02 user-01010

@imtuyethan to answers this question, thank you

It would be nice if we could also edit the response of the assistant. It's a very powerful tool: if the assistant got a few facts wrong, it's easier and more efficient to correct manually the response. Then continue the conversation like normal. It's something I do a lot with llama.cpp server in "Completion" mode, and smaller models really need it to be useful.

I wonder if there's another issue tracking this request or if this isn't currently being considered? Checked the latest nightly and I see we can only edit the user's input.

Van-QA avatar Feb 26 '24 02:02 Van-QA