CRREL / GRiD-API

9 stars 3 forks source link

Inconsistency in primary key data types for POST to /exports #52

Open chambbj opened 3 years ago

chambbj commented 3 years ago

https://github.com/CRREL/GRiD-API/blob/46b84ec55399b969b7f9c918e5779dcef0dd37f7/swagger.v3.yaml#L344-L354

The products parameter must be passed as a string to facilitate comma-separated specification of multiple product exports. On the other hand, the aoi parameter is an integer. Feels awkward and perhaps error prone to sometimes expect integer, other times strings. An alternative could be to pass an array of product pks.