Completes export of versions and external content handling.

fcrepo-exts / fcrepo-import-export

Apache License 2.0

15 stars 19 forks source link

Completes export of versions and external content handling. #122

Closed dbernstein closed 5 years ago

dbernstein commented 5 years ago

All unit tests pass. Only ExporterIT is currently aligned. Other integration tests have been @ignored until they have been brought into alignment.

Resolves: https://jira.duraspace.org/browse/FCREPO-2979

bseeger commented 5 years ago

I'm currently looking through this. Builds fine, but doing a little manual testing.

bseeger commented 5 years ago

I reviewed these changes and, code-wise, they look fine. Things build just fine and an export functions against fcrepo4 code base (master branch). I did notice that a proxied file is exported in whole - which seems like it should not be. I think that's an issue that needs to be talked about more and is on the agenda for the tech meeting on Thursday.

dbernstein commented 5 years ago

@bseeger : I have a feeling the export/import will be a work in progress. I would say merge it if it seems okay for now and we can always create new tickets to address issues that are identified.