Skip to content
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

Silent publishing failure when not all fields required by Datacite are present #7551

Closed
djbrooke opened this issue Jan 27, 2021 · 3 comments · Fixed by #7608
Closed

Silent publishing failure when not all fields required by Datacite are present #7551

djbrooke opened this issue Jan 27, 2021 · 3 comments · Fixed by #7608

Comments

@djbrooke
Copy link
Contributor

djbrooke commented Jan 27, 2021

Short version, this is an attempt to consolidate #6586 and #7518. #7518 has some good logging details and information about the crosswalk.

I'm thinking we'd like to:

  • Have an API endpoint that takes into account PID provider requirements as well as Dataverse requirements when returning a response (basically validate API endpoint doesn't enforce DataCite requirements #6586, just not sure if it should be an existing or new API). There is a wrinkle here that some instances don't use DataCite and it may need to be taken into account when returning what's valid or not, but I'm sure we can work this out.
  • Have better logging at publish time if there's a failure related to a missing field. Whenever this happens it seems like we have to reinvestigate because I forget about this issue. 🤯
  • Better communication to the user when something can't be published because of this (or require it when the Dataset metadata is being added). Right now it just leaves the data in an unpublished state after a silent failure.

Other thoughts welcome...

@djbrooke
Copy link
Contributor Author

  • @scolapasta suggested having both validation and messaging at entry time and at publish time (to catch any that made it through or are existing)
  • In the cases were a subfield is required, there could be a trivial fix - this could be a change to the tsv

@scolapasta scolapasta self-assigned this Jan 27, 2021
@scolapasta scolapasta moved this from Up Next 🛎 to IQSS Team - In Progress 💻 in IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) Jan 27, 2021
@mheppler
Copy link
Contributor

Related? Some Dataverse metadata fields seem not to be harvested/indexed correctly by DataCite #7072

@scolapasta
Copy link
Contributor

Since this is for a specific issue, I went ahead and created a new issue for the general solution of allowing for Conditionally Required fields: #7606

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants