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

API doesn't fail when nameIdentifiers is sent as a object rather the as an array #444

Closed
MaryHirsch opened this issue Jul 3, 2020 · 12 comments · Fixed by datacite/lupo#591
Assignees
Labels

Comments

@MaryHirsch
Copy link

MaryHirsch commented Jul 3, 2020

Expected Behaviour

Access list of DOIs by clicking on DOIs tab

Current Behaviour

DOIs tab blank

Screen Shot 2020-07-03 at 14 26 41

Steps to Reproduce

https://doi.datacite.org/repositories/bf.discover/dois

Context (Environment)

it can be found in the api

https://api.datacite.org/dois?client-id=bf.discover
https://api.datacite.org/clients/bf.discover/dois

image

Fabrica production

Hypothesis

Detailed Description

Possible Implementation

Front logo Front conversations

@kjgarza
Copy link
Contributor

kjgarza commented Jul 6, 2020

maybe related to https://app.frontapp.com/open/msg_e9onoph

@kjgarza
Copy link
Contributor

kjgarza commented Jul 6, 2020

or maybe related to https://app.frontapp.com/open/msg_45cz9mr

@kjgarza
Copy link
Contributor

kjgarza commented Jul 6, 2020

none of the DOIs seem to have a source https://api.datacite.org/clients/bf.discover/dois

@kjgarza
Copy link
Contributor

kjgarza commented Jul 6, 2020

commented all other fragmentArray declaration in the DOI model until i found the one that is failing. that was name_identifiers. For some reasons bf.dissocvery was able to mint doi with an object rather than an array in this field.

// contributors: fragmentArray('contributor', { defaultValue: [] }),

image

@mfenner
Copy link
Contributor

mfenner commented Jul 6, 2020

Good catch @kjgarza! Is nameIdentifiers as object possible with the form or one of the APIs?

And I suggest to update the DOI using the XML upload form.

@kjgarza
Copy link
Contributor

kjgarza commented Jul 6, 2020

The form doesn't allow that. it must have been made with an API. It seems to be many of their dois (or all), Which is strange.

I don't think they can use XML upload to fix it,because one cannot go into the doi page. we will probably need to update them via a rake tasks. They only need a touch to be fixed. there is no need to modify the metadata.

@mfenner
Copy link
Contributor

mfenner commented Jul 6, 2020

Makes sense, thanks.

@jwagenaar
Copy link

Hi all, thanks for looking into this. Copying our engineer @JeremyLecoeur in case you have any questions.

@JeremyLecoeur
Copy link

Hello, I'm the dev responsible on blackfynn side for the DOI requests change that added the name identifier. I see where I made a mistake and I can correct it easily. You were correct in assuming we use the API to update the DOIs. I'm surprised that I did not receive an error code in response since the request was malformed.

@kjgarza
Copy link
Contributor

kjgarza commented Jul 6, 2020

@JeremyLecoeur could you let us know which endpoint did you use? We can review the validation on that endpoint to avoid that problem in the future.

@JeremyLecoeur
Copy link

I used the PATCH https://api.datacite.org/dois/{doi} endpoint

@kjgarza
Copy link
Contributor

kjgarza commented Jul 7, 2020

@JeremyLecoeur could you paste the payload you were sending before ? that would help me to define changes in the validation if needed

@kjgarza kjgarza changed the title DOI list not available in tab for BF.DISCOVER in Fabrica API doesn't fail when nameIdentifiers is sent as a object rather the as an array Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants