Open-Assistant icon indicating copy to clipboard operation
Open-Assistant copied to clipboard

Please add a "Are you Sure you want to Skip" popup to avoid frustration and loss of progress

Open Logophoman opened this issue 2 years ago • 2 comments

Unfortunately I have already written many long answers and while correcting them I have clicked on the skip button which lead to my messages being deleted immediately which is extremely frustrating, especially when you curated that particular answer for half an hour or so already...

If the Assistant or User Message is empty skipping immediately should be no problem, but if there is a message in the response field there should be a popup!

Logophoman avatar Apr 20 '23 12:04 Logophoman

That was old behavior. But I think we can add it back and have a "dont show it again in this session" option and store it in the session storage

notmd avatar Apr 20 '23 13:04 notmd

I think this is the commit that removed the old behavior, but please correct me if I'm wrong.

A possible solution for this would be to bring back the deleted SkipButton, remove the requesting feedback TextArea and just replace it with a Text element to ask a user to confirm if they want to skip or not. I'm less certain on the correct way to interact with the session storage. I see in the existing website code there's reference to useSession from next-auth, but I'm not sure if its best practice to use that to update the session storage for this purpose.

I'm unlikely to work on this issue, but hope this comment helps anyone else interested in working on this!

bterrific2008 avatar May 05 '23 06:05 bterrific2008

I would like to work on this. I had a look at the code, I think it should be enough to just look at the replyValidity, no need to do anything fancy with the session storage

MehrJonas avatar Jun 13 '23 21:06 MehrJonas

@MehrJonas great, I assigned this issue to you. Yes, as you suggested, we don't a "dont show it again in this session" checkbox .. always show the confirmation when the input box is not empty and user tries to skip or simply disable the skip button as long as the text box is not empty.

andreaskoepf avatar Jun 14 '23 20:06 andreaskoepf

#3479

AbdBarho avatar Jun 17 '23 18:06 AbdBarho