search

DigitaleDeltaOrg / dd-grid-api

Ontwikkeling van een Digitale Delta API voor roosterdata
0 stars 1 forks source link

Rename outputCrss endpoint #9

Closed caspervdw closed 3 years ago

caspervdw commented 3 years ago

I find the shorthand Crs in the endpoint and queryparameters a bit awkward. I'd rather rename it to the more readable projection throughout.

Also in the response, we use srsName.

GeriWolters commented 3 years ago

projections (plural) would be more in line with the rest of the API. The name will definitely be more clearer then.

gerritsj commented 3 years ago

I agree that projections is a much clearer name than Crss.

It would be better if the naming in the response and the query are consistent.

stefhummel commented 3 years ago

(During team discussion) Rename outputCrs to projection, both for query parameter in .../data and for /outputCrss end point

stefhummel commented 3 years ago

Currently, the projection of the domainSet of a coverage is still named _srsName (spatial reference system name). Should we rename this attribute to projection, to be consistent with the /projections end point and the projection query parameter?

gerritsj commented 3 years ago

I prefer consistency and I think that projection is more clear than srsName.