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.

Sign up for GitHub

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

Jump to bottom

Fix DDI Import for otherId #10772

Open

luddaniel wants to merge 1 commit into IQSS:develop from Recherche-Data-Gouv:fix_importDDI_otherId

+1 −1

Contributor

luddaniel commented Aug 14, 2024

What this PR does / why we need it:

This PR fixes issue while harvesting DDI with multiple otherId
This fix mirrors citation.tsv otherId configuration : allowmultiples = TRUE

dataverse/scripts/api/data/metadatablocks/citation.tsv

Line 8 in 7d4d534

    
           otherId	Other Identifier	Another unique identifier for the Dataset (e.g. producer's or another repository's identifier)		none	4	:	FALSE	FALSE	TRUE	FALSE	FALSE	FALSE		citation


          Fix MultipleCompoundField for otherId

2ce7b54

luddaniel changed the title ~~Fix MultipleCompoundField for otherId~~ Fix DDI Import for otherId

coveralls commented Aug 14, 2024

coverage: 20.752%. remained the same
when pulling 2ce7b54 on Recherche-Data-Gouv:fix_importDDI_otherId
into 7d4d534 on IQSS:develop.

pdurbin added the Size: 3 label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: 3