-
Notifications
You must be signed in to change notification settings - Fork 2
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
Icebox: Move to new Zenodo API #183
Comments
Automatically adding the |
I notice that we do not list We also don't use |
Ooooh yeah. I noticed that dependency funniness but did not connect the dots with "we don't actually use |
According to email communications with Zenodo this API will not be launched for another year and it's behavior is expected to change. So let's pause on this. |
See also catalyst-cooperative/pudl#2939 -
Zenodo migrated to their new API, we need to use it if we want to keep creating new archives.
f"/records/{record_number}/files/{name}"
in the datapackage.jsonZenodoDepositor
frictionless>=5.0
?Some details on what the "correct endpoints" are: it seems like the basic "make a draft, edit it, publish it" flow is the same as before, which is good.
We have the existing operations defined on our
ZenodoDepositor
:create_deposition
- now create a draft recordget_deposition
- this gets the latest record associated with a concept DOI, we need to test if get a record works with concept DOIsget_record
- this just gets the single record associated with a DOI - so just get a recordget_new_version
: this makes a new draft, copies the files, and updates the version. Maybe we want to break out the end point hits in here into their own functions.edit a published record
actually makes a snapshot like we expect.publish_deposition
: publish a draft recorddelete_deposition
: delete / discard a draft recordcreate_file
: start draft file upload(s), upload a draft file's content, complete draft file uploaddelete_file
: delete a draft fileupdate_file
: this one is defined in terms ofcreate_file
andupdate_file
so I think we're good.Sandbox changes:
5281
prefix, and all sandbox archive DOI values have changed. These will need to be updated.The text was updated successfully, but these errors were encountered: