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

Incorporate Bulkrax x Valkyrie changes #97

Closed
Tracked by #93 ...
ShanaLMoore opened this issue Aug 24, 2023 · 0 comments
Closed
Tracked by #93 ...

Incorporate Bulkrax x Valkyrie changes #97

ShanaLMoore opened this issue Aug 24, 2023 · 0 comments
Assignees

Comments

@ShanaLMoore
Copy link

ShanaLMoore commented Aug 24, 2023

Kiah started a PR to get Bulkrax working with Valkyrie. We will want to pull in those changes and confirm that it we can successfully import works.

ref: samvera/bulkrax#818

For context, we're prioritizing the XML parser.

Separate tickets can be made to handle the XML/Manifest and CSV Parser if they are currently not working.

This was referenced Aug 24, 2023
@ShanaLMoore ShanaLMoore changed the title Upgrade Bulkrax again, After Hyrax Upgrade Incorporate Bulkrax x Valkyrie changes Aug 29, 2023
@jillpe jillpe added this to the ActiveFedora migration milestone Aug 29, 2023
@ShanaLMoore ShanaLMoore self-assigned this Aug 30, 2023
ShanaLMoore added a commit to WGBH-MLA/ams that referenced this issue Aug 30, 2023
This work is still in progress. The valkyrie config file is commented out until we receive code for implementing adapters. We are using the object_factory config to utilize Valkyrie::ObjectFactory and lastly we added some changed to avoid errors when running an import.

Issue:
- scientist-softserv#97

Co-Authored-By: Kirk Wang <k3wang@gmail.com>
Co-Authored-By: LaRita Robinson <laritakr@users.noreply.github.com>
@jillpe jillpe closed this as completed Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

4 participants