Skip to content
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

Work status "Deposited" showing in work view #3107

Open
julesies opened this issue Feb 22, 2017 · 4 comments
Open

Work status "Deposited" showing in work view #3107

julesies opened this issue Feb 22, 2017 · 4 comments

Comments

@julesies
Copy link

7.3.0rc1

Status (in workflow?) is being displayed on the work view, right under the work title. This is happening when I am logged in and not logged in.
screen shot 2017-02-22 at 3 20 49 pm

I would not expect the deposit status to show for a regular "viewer" of any work view.

Disclaimer, we are still testing workflows and haven't really got it working, but this happens even to items that are going into the default admin set, which doesn't have an assigned workflow. We upgraded from 7.2.

@mjgiarlo
Copy link
Member

@julesies Hi, we do have a ticket for making the workflow status more polished and clearer. I'm not sure we have consensus yet on whether this should display or not, or what users this should display to, so for now I am inclined to leave this displaying all the time if that's OK. (And, in the meantime, I'd love for someone to spiffy the design up a bit so it doesn't look... vestigial.) How does that sound? (If cool, feel free to close the issue. If not, it may be good for community discussion.)

@julesies
Copy link
Author

julesies commented Feb 27, 2017

I can certainly close it. We aren't using 7.3 since we would start fresh on hyrax, just testing. For the future plans, I would worry more about what context it shows and to who the status shows. I have found mediated deposit is messy and riddled with mistakes and do-overs. So those papers that are in a review process, really shouldn't show to the public at all until it is properly vetted. Perhaps the fact that I'm seeing this when logged out (public view) is related to other bugs or config issues we have going on ❓

@mjgiarlo
Copy link
Member

@julesies Any works that are under review should only be visible to:

  • A repository administrator
  • A manager for the admin set into which the work was deposited
  • A reviewer for the admin set into which the work was deposited
  • A user with the approving role in the mediated deposit workflow

If they are visible to logged-out users, that is definitely a bug. (And one that I suspect we've already fixed in Hyrax, which could be backported to Sufia.)

@julesies
Copy link
Author

julesies commented Mar 13, 2017

@mjgiarlo this is still happening on rc3, except now only after the review has been approved. This screenshot shows "complete" status. Notice this is the "logged out/public view".
screen shot 2017-03-13 at 10 09 20 am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants