ucsdlib / damsmanager

DAMS Manager
Other
3 stars 1 forks source link

Add multiple values support for header Identifier:identifier #258

Closed lsitu closed 6 years ago

lsitu commented 6 years ago

Descriptive summary

Add support for header Identifier:identifier with multiple columns or delimited by | character for multiple values.

Rationale

Currently, DAMS Manager assumes header Identifier:identifier to be single value and will merge multiple Identifier:identifier columns together into a single identifier note. While in object https://library.ucsd.edu/dc/object/bb0515927k, it has multiple values for the identifier note. We need to separate them.

Expected behavior

Multiple columns of Identifier:identifier should be separated with multiple Identifier notes.

Actual behavior

Multiple Identifier:identifier columns is merge together into a single Identifier note delimited by | character.

Related work

251

lsitu commented 6 years ago

@mcritchlow / @remerjohnson I've added PR https://github.com/ucsdlib/damsmanager/pull/259 for the support.

gamontoya commented 6 years ago

@remerjohnson or @GregReser This ticket needs confirmation that this is working on staging.

gamontoya commented 6 years ago

@remerjohnson Can you confirm this one is working as expected?

remerjohnson commented 6 years ago

@gamontoya Yup! Confirmed this as working on staging