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.

Sign up for GitHub

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

Jump to bottom

Enqueues Job that wasn't calling. #6478

Merged
merged 1 commit into from Nov 30, 2023
Merged

Enqueues Job that wasn't calling. #6478

merged 1 commit into from Nov 30, 2023

Conversation

bwatson78
Copy link
Contributor

Fixes

Fixes spec/controllers/hyrax/file_sets_controller_spec.rb.

Summary

Enqueues Job that wasn't calling.

Type of change (for release notes)

  • notes-valkyrie Valkyrie Progress

@samvera/hyrax-code-reviewers

@bwatson78 bwatson78 added valkyrization notes-valkyrie Release Notes: Valkyrie specific labels Nov 28, 2023
@bwatson78 bwatson78 self-assigned this Nov 28, 2023
dlpierce
dlpierce approved these changes Nov 30, 2023
View reviewed changes
Copy link
Contributor

@dlpierce dlpierce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm curious what causes the number of run calls to decrease from this change.

@dlpierce dlpierce merged commit 7ef145e into main Nov 30, 2023
4 checks passed
@dlpierce dlpierce deleted the updates_file_sets_controller_spec branch November 30, 2023 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dlpierce dlpierce dlpierce approved these changes

Assignees

@bwatson78 bwatson78

Labels
notes-valkyrie Release Notes: Valkyrie specific valkyrization
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bwatson78 @dlpierce