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

RepositoryAuditService audits not showing results in the UI #188

Closed
mjgiarlo opened this issue Dec 21, 2016 · 5 comments · Fixed by #984
Closed

RepositoryAuditService audits not showing results in the UI #188

mjgiarlo opened this issue Dec 21, 2016 · 5 comments · Fixed by #984
Assignees
Labels
Milestone

Comments

@mjgiarlo
Copy link
Member

Issue by electrickite
Tuesday Nov 15, 2016 at 14:42 GMT
Originally opened as samvera-deprecated/sufia#2886


Descriptive summary

When running audits using Sufia::RepositoryAuditService.audit_everything, the results are not shown in the UI.

Observed in Sufia 7.0.0, but I can't see anything in later versions that would have changed this behavior.

Expected behavior

Running audits on a FileSet will show the results of the audit on the file set show page.

Actual behavior

The audit status remains 'Audits have not yet been run on this file.' even if audits have been run.

Steps to reproduce the behavior

  1. Create at least one Work with an attached FileSet
  2. From a Rails console, run Sufia::RepositoryAuditService.audit_everything
  3. If needed, wait for background audit jobs to complete.
  4. From a Rails console, run ChecksumAuditLog.all and observe that audit log entries have been created.
  5. Navigate to the FileSet "show" page and observe the audit status is still 'Audits have not yet been run on this file.'
@mjgiarlo mjgiarlo added this to the Backlog milestone Dec 21, 2016
@mjgiarlo
Copy link
Member Author

Comment by mjgiarlo
Tuesday Nov 15, 2016 at 15:04 GMT


Thanks, @electrickite! Do you have cycles to troubleshoot?

@mjgiarlo
Copy link
Member Author

Comment by mjgiarlo
Tuesday Nov 15, 2016 at 15:05 GMT


Related to samvera-deprecated/curation_concerns#1091

@mjgiarlo
Copy link
Member Author

Comment by electrickite
Tuesday Nov 15, 2016 at 15:08 GMT


@mjgiarlo I should be able to track this down a little more later today or tomorrow. I would not be surprised to find out this is bubbling up from curation concerns, but I figured we should track it in Sufia since it affects Sufia components.

@mjgiarlo
Copy link
Member Author

Comment by mjgiarlo
Friday Dec 16, 2016 at 23:27 GMT


@electrickite have you had a chance to look into this?

@mjgiarlo
Copy link
Member Author

Comment by mjgiarlo
Tuesday Dec 20, 2016 at 21:49 GMT


I suggest moving this work to Hyrax vs. it blocking the 7.3.0 release. If someone is inclined to put in a patch before the 7.3.0 release is cut, I am happy to include this in the release.

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

Successfully merging a pull request may close this issue.

3 participants