search

clowder-framework / pyclowder

Library to assist in the development of extractors for clowder.
University of Illinois/NCSA Open Source License
10 stars 2 forks source link

Fixing delete method #63

Closed tcnichol closed 1 year ago

tcnichol commented 1 year ago

The delete method was throwing a 400 error. The reason is that the headers={contentType=application.json} was not correct for delete endpoints in clowder. This is fixed.

tcnichol commented 1 year ago

I noticed that there was a place where v2 delete was used instead of v1 in datasets.py. I fixed that on this branch as well.

Also, looks like with the changes the comment from Rob says outdated, but I want to make sure that this works with an api key. Or would be always now be using Client so which would have a key so it's no longer necessary?

max-zilla commented 1 year ago

@tcnichol please resolve merge conflict.

tcnichol commented 1 year ago

Merged and fixed the merge conflict.