Add entry for Broad PGM API

NCATS-Tangerine / translator-api-registry

This repo hosts the API metadata for the Translator project

13 stars 31 forks source link

Add entry for Broad PGM API #3

Closed cmungall closed 7 years ago

cmungall commented 7 years ago

cc @vdancik

newgene commented 7 years ago

@vdancik thanks for submitting the PGM API to the registry!

There are a few issues to be fixed in order to have it pass the validation:

Fix the mis-match in the entry from API_LIST.yml and the actual file name (openapi.yaml v.s. openapi.yml)
Add a required "translator" tag. See the example here.
Add a required github link for the responsible dev. See the example here.

Other than that, it looks good to me.

vdancik commented 7 years ago

@newgene thanks for the comments, they were addressed in the updated version.

newgene commented 7 years ago

@vdancik thanks. PGM API is now on SmartAPI registry! http://smart-api.info/registry

Though optional, it will be nice to provide an "info.description" field (supports MarkDown) for your API, like other APIs, e.g. pointing to the website. It also renders better in the registry.