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

Sirenia testing - VR_1.2 test: error when approving or requesting changes on a 1 step workflow deposit #6521

Closed
2 tasks
eporter23 opened this issue Dec 8, 2023 · 2 comments

Comments

@eporter23
Copy link
Contributor

eporter23 commented Dec 8, 2023

Descriptive summary

When an Admin approves a deposit submitted through a basic 1-step workflow, it returns an Valkyrie::Persistence::ObjectNotFoundError in Hyrax::WorkflowActionsController#update error. Same error occurs if trying to request revisions on the work or if trying to Edit the work as an admin. The Dashboard notifications do show that the approval and review changes have occurred.

Steps to reproduce the behavior in User Interface (UI)

  1. Create an Admin Set allowing 1-step workflow, and all visibility and release options (see test VR_1.2)
  2. As a different user, create a new work and make sure it's assigned to this Admin Set
  3. As an approving/admin user, go to review the new work
  4. Fill out the approval options on the work and submit
  5. See the below error message

Actual behavior (include screenshots if available)

Include what version of Hyrax relates to this issue (3.x, 4.x, main branch, etc.) if appropriate, and any relevant error messages/tracebacks if you're reporting a bug.

Hyrax 5rc2 main, Sirenia (Fedora 6) docker image

Screen Shot 2023-12-07 at 2 14 20 PM

Note: the approval and/or request revisions activity does show up in the users' notifications:

Screen Shot 2023-12-07 at 2 17 01 PM

Acceptance Criteria/Expected Behavior

  • When a user with appropriate permissions is shown the option to Approve a 1-step deposit submission, they should not see an error message, and should be returned to the work with the appropriate status indicated
  • The work should be transitioned to the Published/Deposited state

Rationale (for feature request only)

Provide the rationale or user story that describes "why" this issue should be addressed. Especially if this is a new feature or significant change to the existing implementation.

1-step workflows should be supported in Fedora 6-backed repositories

Related work

Link to related issues or prior related work here.

@eporter23
Copy link
Contributor Author

In recent local testing at Emory, we are no longer seeing this issue (Hyrax 5.0.1, Fedora 6.5) so this may be closable after retesting by others.

@jlhardes
Copy link
Contributor

I tried this out on Hyrax 5.0.1 locally with Sirenia and I cannot come up with an error. I set up the mediated deposit admin set and then as a non-admin user, I created a private work in that admin set. I approved it as the admin user and it worked. Then I created a public work as the non-admin user in that admin set and proceeded to have a review conversation between the admin user and the non-admin user about changes that were needed. The non-admin user made the changes and the admin user approved the deposit and it all worked. I could get to the work through the notifications links without any problems. I think this issue is resolved.

@rjkati rjkati closed this as completed May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants