search

NASA-PDS / pds-api

PDS web APIs specifications and user's manual
http://nasa-pds.github.io/pds-api
Other
5 stars 3 forks source link

Replace summary-only=true by limit=0 #173

Closed tloubrieu-jpl closed 2 years ago

tloubrieu-jpl commented 2 years ago

💪 Motivation

...so that we don't have this extra parameter. This is also consistent with elasticSearch or solr API behavior for example.

⚙️ Engineering Details

jordanpadams commented 2 years ago

@tloubrieu-jpl so if I understand this update, we just want to remove the summary-only parameter?

al-niessner commented 2 years ago

Done with PR NASA-PDS/registry-api#166

jordanpadams commented 2 years ago

Reverting per issues identified with #179 and difficulty in making it work for all interfaces

jimmie commented 2 years ago

Worked out a way to do this across the board w/ limit=0. Not the best solution, but it works for now.

jimmie commented 2 years ago

PR approved and merged