-
Notifications
You must be signed in to change notification settings - Fork 451
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
Redesigned Workflow - FileManager #10688
Comments
@jardakotesovec this passes QA and can be closed. I tried to limit the scope. You have addressed all the major issues for UI (See screenshot 1) and the functionality works as expected. Hard to say for certain as there are several scenarios that I have no clue what the existing behavior was. Search functionality was skipped General tests have been created for the following areas. Let me know if there is something I missed.
One potential issue is upload of files not being seen on revision list for stages that have already been passed. Solution and enhancement would be to remove the ability to add files after a stage has been completed? Currently this is similar functionality as expected on older OJS versions. |
FileManager is replacing the legacy file grid used in 3.4. But new UI is only for listing the files. All modals are exactly same as before.
Differences to 3.4
QA Notes:
What to look for in terms of regressions:
Reviewer's Attachments
Reviewer's attachments is file grid displayed to author in review stage
It has been bit mystery to me how that works, so will try to cover it here.
In 3.3 when Editor would request revisions from Author - its possible to select files that has been submitted/revised but also upload new files. My understanding is that the problem was only the existing ones would be shared via Reviewer's attachment grid and other would be included only as email attachment, which was confusing - its covered more in detail here #6805 . From technical point of view 3.3 was setting such files as 'viewable', which was used to determine which files to show in Reviewer's Attachment grid. Viewable also got deprecated in 3.3.
And with 3.4 and later this approach seems to be abandoned as its using new editorial decision to request revisions with new UI. All files gets attached to the email, but did not manage to display anything in
Reviewer's Attachment grid.
. Which might be better to show nothing than just some of them as described above.From the issue (https://github.com/pkp/pkp-lib/issues/7144) we can conclude its desirable to be able to see these files on author dashboard, but it would require notable effort to get that working, therefore keeping it as future improvement request.
In context of my current effort it seems that there is no way to display files in Reviewer's Attachment grid in 3.4/3.5.. so I will not include it in new workflow page.
Tagging @asmecher @Devika008 @Vitaliy-1 For awareness on this. If you think I made some incorrect conclusion let me now.
The text was updated successfully, but these errors were encountered: