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

User guide DOI #177

Closed
wants to merge 1 commit into from
Closed

Conversation

tdddblog
Copy link

@tdddblog tdddblog commented Apr 30, 2022

🗒️ Summary

Cookbook section and instructions on searching for DOI in product's parent collection and bundle

⚙️ Test Data and/or Report

Read "Cookbook" section (09f70e1)

NOTE: This is a sub-branch of another non-main branch.

♻️ Related Issues

#89

@tdddblog tdddblog requested a review from a team as a code owner April 30, 2022 00:13
@jordanpadams jordanpadams changed the base branch from main to pds-api-169-api-user-guide May 3, 2022 16:58

.. code-block:: python

curl -X GET --header 'Accept: application/json' 'http://localhost:8080/products/urn%3Anasa%3Apds%3Akaguya_grs_spectra%3Adata_ephemerides%3Akgrs_ephemerides%3A%3A1.0/bundles?limit=100&fields=ops%3AIdentifiers%2Fops%3Adoi&fields=pds%3ACitation_Information%2Fpds%3Adoi&only-summary=false'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tdddblog can we change localhost:8080 to the URL where the API will reside?

@tloubrieu-jpl can you provide that URL? pds.nasa.gov/api/v1.0 ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jordanpadams I will do the change when I merge

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jordanpadams @jimmie @tdddblog I can not find a working case for the DOI request in the cookbook. If we are not able to provide that in our registry servers, I believe it is better to table the publication of that cookbook.

@jordanpadams
Copy link
Member

Moved to other PR: 09f70e1

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 this pull request may close these issues.

3 participants