Another hefty set of changes in this PR, apologies in advance π
This PR modifies the available actions in the DOI workflow to add an update action to take the place of draft. The update action may be used to update the metadata fields, including PDS identifier, for any record with a DOI already assigned to it. Any requests without an associated DOI are rejected by the update action.
This branch also introduces a number of updates/fixes to support the new update action, including:
Support for parsing several optional columns from an input spreadsheet, such as DOI and site URL. This helps enable batch updates to existing records.
Support for keeping a "rolling" list of related identifier fields in order to keep track of all LIDVIDs/PDS3 identifiers ever associated to a record.
Better validation of records, with validation tailored to the specific action being performed
Addition of convenience functions to the list action for obtaining the latest transaction and label from the local database for a given DOI or identifier.
βοΈ Test Data and/or Report
Unit tests have been added for the update request, as well as for some other utilities that never had a proper test suite (like pds4_util.py). Existing unit tests have also been updated to accommodate
tox.log
the changes made by this PR.
ποΈ Summary
Another hefty set of changes in this PR, apologies in advance π
This PR modifies the available actions in the DOI workflow to add an
update
action to take the place ofdraft
. The update action may be used to update the metadata fields, including PDS identifier, for any record with a DOI already assigned to it. Any requests without an associated DOI are rejected by the update action.This branch also introduces a number of updates/fixes to support the new update action, including:
βοΈ Test Data and/or Report
Unit tests have been added for the update request, as well as for some other utilities that never had a proper test suite (like
pds4_util.py
). Existing unit tests have also been updated to accommodate tox.log the changes made by this PR.β»οΈ Related Issues
Resolves #278 Resolves #279