Swirrl / datahost-prototypes

Eclipse Public License 1.0
0 stars 0 forks source link

ldapi: rename change releated routes #241

Closed rosado closed 1 year ago

rosado commented 1 year ago

We have the following routes

POST /data/.../:revision-id/changes
GET  /data/.../:revision-id/changes/:change-id
POST /data/.../:revision-id/deletes

(And I'm about to add a POST .../corrections)

This PR renames two routes to be consistent with our internal naming (:datahost.change/kind -> #{:dh/ChangeKindAppend :dh/ChangeKindRetract}):

POST /data/.../:revision-id/appends
POST /data/.../:revision-id/retractions

This would leave us with GET .../changes. The only issue is we don't allow getting the metadata of a change (only CSVs at the moment), so querying that endpoint won't tell us whether the returned CSV is an append, retraction or correction. The user is expected to get that information via another call (see here)