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

Prompt for reason for skipping.

Open othrayte opened this issue 2 years ago • 2 comments

This is a draft for #310 (Add ability to tag tasks as unanswerable).

It tasks onboard the feedback that we want to guide people away from skipping a bit and that we want to know why they have skipped. The leading idea for that appeared to be allowing the user to provide a custom reason, so that is the direction this went.

The main reason that this is only a draft PR is that at the moment we are not able to NACK after ACK-ing and the website auto-ACKs on fetching tasks so there is no chance to NACK. In this PR I have commented out the auto-ACK-ing to allow for testing this.

othrayte avatar Jan 08 '23 07:01 othrayte

I've moved the ACK to just before submitting interactions. Also I've fixed the summerisation and labeling tasks temporarily by making them skip without NACK-ing as they don't yet use the new Task.tsx component and I wasn't sure if someone was already working on doing that and figured that would be the better way to handle them.

othrayte avatar Jan 08 '23 13:01 othrayte

Apologies for any review request spam, I'm still learning how that action works on github.

othrayte avatar Jan 08 '23 13:01 othrayte

Looks good, we can fix any small issues later.

fozziethebeat avatar Jan 08 '23 23:01 fozziethebeat