Closed alexdunnjpl closed 1 year ago
@alexdunnjpl @nutjob4life holding on merging this for the time being. awaiting some feedback from our PDS DOI expert to ensure we are doing this right :-)
@alexdunnjpl you may be able to just add the licensing metadata update to this PR in the meantime.
Behaviour as-implemented:
release
applied with no rightsList
attribute present in label data gets GWorks and CC0release
applied with empty or other-valued rightsList
attribute in label data gets whatever the label specifies.
@jordanpadams ready to merge, pending your DOI MVP A-OK :sweat_smile:
@alexdunnjpl do we need to update the test data that is out there?
@alexdunnjpl do we need to update the test data that is out there?
@jordanpadams I'm not sure. No tests failed, so I'm not sure if it's within the scope of existing tests. I'll take a closer look at that test data early next week.
merging. will poke at test data if anything breaks.
@jordanpadams I can't turn up any string references to these files and the commit histories are intractably-large in terms of identifying related files/modules/tests.
Given that tests aren't failing, I'd suggest it's not worth investigating further.
🗒️ Summary
Implements a rights-management statement class, adds rights management (
rightsList
) metadata to DataCite reservations andDOIDataCiteRecord
s, and sets rights-management metadata to a US government-works copyright, by default⚙️ Test Data and/or Report
All tests pass except flake8 lint, no tests added.
♻️ Related Issues
fixes #335