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

Add section on special bag structures #354

Merged
merged 1 commit into from
Feb 10, 2020
Merged

Add section on special bag structures #354

merged 1 commit into from
Feb 10, 2020

Conversation

sallain
Copy link
Member

@sallain sallain commented Feb 10, 2020

I've added a section about alternate configurations for bags. It's a bit
of a punt, as we don't test all of the possible bag configurations, but
I'm not sure how else to communicate that there may be other setup
options out there while also acknowledging that A) Artefactual doesn't
test them all, B) some of them might not work, and C) if they don't work
it doesn't mean it's a bug. So this is what I've landed on.

Connected to archivematica/Issues#669

I've added a section about alternate configurations for bags. It's a bit
of a punt, as we don't test all of the possible bag configurations, but
I'm not sure how else to communicate that there may be other setup
options out there while also acknowledging that A) Artefactual doesn't
test them all, B) some of them might not work, and C) if they don't work
it doesn't mean it's a bug. So this is what I've landed on.

Connected to archivematica/Issues#669
@sallain sallain requested a review from ablwr February 10, 2020 22:19
@sallain sallain merged commit d39af3d into 1.11 Feb 10, 2020
@sallain sallain deleted the dev/special-bags branch February 10, 2020 23:50
@ross-spencer
Copy link
Contributor

Late to the 🎉 on this one, but to the extent I can help with testing of bags, and in future testing AMAUAT coverage, there's a draft PR here which brings in a custom processing config in one transfer, and a DemoTransferCSv type bag, with the paths updated to work correctly when processed. Anyway, hope it's helpful.

@sallain
Copy link
Member Author

sallain commented Feb 11, 2020

Thanks @ross-spencer! Those are great to have.

@ross-spencer
Copy link
Contributor

I was hoping! And I meant to say as I introduced that PR, these are also good docs. Good to see them.

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.

3 participants