datatracker icon indicating copy to clipboard operation
datatracker copied to clipboard

When assignee rejects a review request, it seems to mark him for next for assignment

Open kivinen opened this issue 2 years ago • 3 comments

Describe the issue

When someone rejects an assignment, for example because they are too busy now, the system seems to automatically mark that parson to "someone who has indicated he wants to be next in queue", which they definately have not done.

As a secretary I always need to go and edit those users information and remove that checkbox from them before I can continue my assignments, as otherwise it will try to assign next document to them, which they will reject as they are still busy etc.

Code of Conduct

kivinen avatar Sep 25 '22 06:09 kivinen

The reject workflow may need to have an explicit "why" added if it's not already there. This was added because of people rejecting documents because they had a conflict of interest that the review system didn't already detect, and they should go review something else instead.

That said, People who are too busy should be marking themselves as unavailable (or you should do this for them for the period that they are too busy for.

rjsparks avatar Sep 26 '22 13:09 rjsparks

Robert Sparks writes:

The reject workflow may need to have an explicit "why" added if it's not already there. This was added because of people rejecting documents because they had a conflict of interest that the review system didn't already detect, and they should go review something else instead.

That has not happened ever on secdir. All rejections have happened because people suddenly find out that they are too busy to do review, especially if the document they got to be reviewed happened to be long...

That said, People who are too busy should be marking themselves as unavailable (or you should do this for them for the period that they are too busy for.

Yes, but usually they notice they are too busy to do reviews after they get review assigned... Also as they only get review requests every 2-3 months, they might not want to bother to mark themselves unavailable for every single one or two week time.

@.***

kivinen avatar Sep 27 '22 20:09 kivinen

It does happen at genart, and a radio button at the reject step would let the right thing happen for either use case.

rjsparks avatar Sep 27 '22 20:09 rjsparks

Checking this, it seems to mark them to be next in queue ONLY if you are using the RotateAlphabeticallyReviewerQueuePolicy.

LeastRecentlyUsedReviewerQueuePolicy will simply ignore the rejected/withdrawn reviews. Is the genart also using the rotate alphabetically policy?

kivinen avatar Mar 25 '23 02:03 kivinen

genart is using least recently used

rjsparks avatar Mar 25 '23 04:03 rjsparks