-
Notifications
You must be signed in to change notification settings - Fork 493
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
"Production date" hard coded in JSON metadata export #6426
Comments
@jggautier good catch. Here's where it seems to be hard coded: |
@jggautier Thanks. So a quick fix (hopefully :)) and a note about running reexports in the release notes. |
@jggautier @scolapasta - can you discuss before we pick this up for the sprint? Thank you! |
The to-do in the code that @pdurbin pointed to says "get "Production Date" from datasetfieldvalue table": I didn't realize that when the dataset also has a user-entered Distribution Date, the JSON export also includes the distributionDate at the "datasetVersion" level: So it seems like the intent was to repeat, in the JSON export, the date metadata that users enter in the Distribution Date and Production Date fields (in the Citation metadata block). (But right now the Production Date is hard coded, and that to-do is saying that it should be dynamic, like the Production Date is.) But why is the dataset version's Distribution Date and Production Date metadata, already in the JSON export at the "metadataBlocks" level, repeated at the "datasetVersion" level above it? Is it because it's easier to get the values at that level? Why not include the Deposit Date? |
Adding myself to add a release note for re-running exports. |
In recent versions of Dataverse, including the latest 4.18.1, the"JSON" metadata export of every dataset includes
productionDate: "Production Date"
.It seems to be hard coded, appears only in the "JSON" export, and doesn't seem to affect how the "Production Date" metadata (that depositors fill out) is recorded (see
typeName: productionDate
at the bottom of the screenshot).But it could confuse people who are trying to make sense of the metadata in the JSON export and I think should be removed. Then I think each installation will need to re-publish each published dataset's JSON metadata export.
The text was updated successfully, but these errors were encountered: