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
Editorial history should save the datestamp when a new review round starts #4042
Comments
Ok, so when you start a new review round, the editorial history says "An editor decision (Resubmit for Review) for article 48 was recorded by admin admin." which is the exact same entry as the one you get when you actually record a decision like that. This is a bug right? I think it would make sense if starting a new review round would record "A new review round (#) was created by admin admin." Starting a new round does not send a resubmit for review message to author, right? |
@NateWr what do you think? I could do a quick pr for the new message if you agree. |
Hmm, I am not sure. As I understood it, starting a new review round was meant to be equivalent to resubmit for review. But I'm not certain of that. I think we'll need to call on @asmecher. (In general I'm in favor of recording the information, just want to clarify how the underlying data differs between the two routes of opening a new round.) |
That does not sound right, doing a "resubmit for review" decision and starting a new review round are really two totally different things. When an editor does a "resubmit for review" decision on Round 1, the editor explains to the author the changes needed based on review reports from round 1. The author then sends the new version of the manuscript including those changes to the Revisions box in round 1 and after that the editor starts a new review round using the changed manuscript (however, now the editorial history shows two "Resubmit for Review" decisions which does not make sense => the author has already resubmitted and a new review round has already started). This is also indicated in Learning OJS although there is no example for a two round review: https://docs.pkp.sfu.ca/learning-ojs/en/editorial-workflow#review The editorial decision "resubmit for review" does not open a new round. It is just an indication that major revisions are needed, just a label really. Even if I do a "resubmit for review" decision, I can still just accept the changed manuscript without starting a new round. The bottom line: "resubmit for review" decisions are always made before a new review round is started and the process is: Resubmit for review decision > Author sends a revision > A new review round is created. |
My understanding is that "revisions requested" should be used when revisions are not expected to require another round of review. The ambiguity between these two is obviously a source of confusion. |
Yes "revisions requested" is usually used if no new review round is expected. I just wanted to underline that the decision there is just a label. If I do a "revisions requested" decision, I can still start a new review round when the revisions arrive If I would decide, I would change the labels to "Minor revisions" and "Major revisions". But that is not the point here. The important thing would be that the editorial history makes a clear difference between the "Resubmit for review" decision and the event where the new review round is actually started. |
@asmecher do you have time to check this. If I am wrong on the issue, then I have to quickly revise the translation for the Learning OJS guide and I think the original English version also needs some clarification (I think having the whole Round 2 process explained in the guide would be a good thing). |
sorry for bugging you @asmecher but I am planning to continue translating the Learning OJS3 guide for the editors and having the above question solved would be nice... |
Sorry for the delay, @ajnyga! I agree with your first proposal, to log a "new review round" message that's distinct from "revisions requested". The major/minor distinction could be useful, but I'm hesitant to introduce new language when it's not used elsewhere. |
Hi, the major/minor was really a secondary issue here. The main point is definitely that starting a new review round != resubmit for review decision, because that decision should have already taken place before the new review round is created. I will make a pr where I suggest a new editor decision "SUBMISSION_EDITOR_DECISION_NEWROUND" and we can continue the discussion based on that. |
@asmecher this is probably not solved yet? I see someone promised a pr back in sep 2018, but has not delivered... |
I don't think anyone has stepped on someone's toes yet :) |
Can you assign this to me so I fo not forget! |
Done, thanks! |
@asmecher If that looks good, I will prepare a pr for OMP as well. OPS does not require this. |
Thanks, @ajnyga, this looks good -- please prepare an OMP PR and I'll merge all three. |
omp: pkp/omp#794 |
@asmecher one test failed here: pkp/omp#794 if you can restart |
^ Restarted and passed! Is this ready for merge, @ajnyga? |
Yes, this is all ready |
At the moment it is hard to determine what has happened during review round 1 and what during review round 2.
The text was updated successfully, but these errors were encountered: