samvera / bulkrax

Bulk Import and Export For Samvera
Apache License 2.0
12 stars 21 forks source link

I1010 embargo lease support #950

Closed ShanaLMoore closed 5 months ago

ShanaLMoore commented 5 months ago

Summary

Changes c/o PITT.

We are adding support to enable users to set embargo and lease rules from a CSV import.

Acceptance Criteria

Sample CSV:

embargo-lease.csv

Screenshots / Video

The following works' embargo/lease rules were created by uploading the sample CSV.

Screenshot 2024-04-16 at 17-40-38 LEASE set via Bulkrax Screenshot 2024-04-16 at 17-40-31 Show Entry __ Hyku Screenshot 2024-04-16 at 17-40-27 EMBARGO set via Bulkrax Screenshot 2024-04-16 at 17-40-19 Show Entry __ Hyku Screenshot 2024-04-16 at 17-39-26 Show Importer __ Hyku

Raise error if model is not valid, which previously used to error out with:

*** NoMethodError Exception: undefined method `method_defined?' for nil:NilClass

Screenshot 2024-04-16 at 17-36-28 Show Entry __ Hyku

Expected Behavior

Notes

TODO: Update bulkrax wiki with documentation once this gets merged

kerchner commented 5 months ago

😃 👍 Very happy to see this work. We generally embargo just the FileSets but not the parent work, so I assume that by specifying the parent work and its FileSets on separate lines of the CSV, that this will enable embargoing each piece separately