search

nytimes / public_api_specs

The API Specs (in OpenAPI/Swagger) for the APIs available from developer.nytimes.com
http://developer.nytimes.com
Apache License 2.0
136 stars 40 forks source link

Several API spec discrepancies #49

Closed leslie-heinzen closed 5 years ago

leslie-heinzen commented 5 years ago

Hello,

Thanks for making these API specs publicly available.

I have found several discrepancies between the API specs available in this repo and those served from the developer portal. For example, comparing the article search api spec here and a parsed snapshot of the one served from the developer portal:

There are many diffs -- the one in this repo appears to be v1. Can I assume that the specs served from the developer portal are the source of truth? Cursory testing seems to indicate this. If this is the case I would be happy to make some PRs to update the specs here.

Thanks for your time.

nyt-hughmandeville commented 5 years ago

The specs in the developer portal are updated versions of those in the public_api_specs repo.

I checked in the dev portal versions into public_api_specs. They are YAML files instead of JSON files (e.g. article_search_v2.yaml). We'll remove the old JSON versions. Thanks