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

🐛 Fix Bulkrax Collection Relationships job #199

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

laritakr
Copy link
Contributor

Story

Importing a work with a collection did not add the work to the collection. By moving the Hyrax::BasicMetadata include below the property definition, the relationship job now completes normally and makes the appropriate relationships.

refs

Expected Behavior Before Changes

Works were not added to collections via a bulkrax import.

Expected Behavior After Changes

Adding a collection ID to the parents column will create the work's relationship to the appropriate collection.

Screenshots / Video

Screenshot 2023-11-27 at 4 21 48 PM

Notes

@laritakr laritakr force-pushed the fix-bulkrax-collection-relationship branch from c289a93 to 38a7265 Compare November 27, 2023 21:31
@laritakr laritakr force-pushed the fix-bulkrax-collection-relationship branch from 38a7265 to 294437d Compare November 27, 2023 21:53
Importing a work with a collection did not add the work to the
collection. By moving the Hyrax::BasicMetadata include below the
property definition, the relationship job now completes normally and
makes the appropriate relationships.

refs
- #195
@laritakr laritakr force-pushed the fix-bulkrax-collection-relationship branch from 294437d to 51b62bc Compare November 27, 2023 22:04
@laritakr laritakr merged commit f98d95f into main Nov 28, 2023
4 of 7 checks passed
@laritakr laritakr deleted the fix-bulkrax-collection-relationship branch November 28, 2023 00:35
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

Successfully merging this pull request may close these issues.

2 participants