search

GALAglobal / TAPICC-API-implementation

TAPICC API implementation using node.js framework sails.js
Other
6 stars 1 forks source link

Plural for URI #10

Closed Alino closed 6 years ago

Alino commented 6 years ago

From TAPICC created by ysavourel : GALAglobal/TAPICC#13

One suggestion: Using the plural forms for the URIs:

/jobs
/jobs/{id}
/jobs/{jobId}/assets
/jobs/{jobId}/assets/{assetId}
etc.

It feels more intuitive (GET /jobs lists all the jobs, GET /jobs/123 gets the job 123 among all jobs)

Alino commented 6 years ago

I think this should be implemented, because top tech companies are using this approach.

ysavourel commented 6 years ago

+1 for plural as it seems to be the most common convention across many APIs.

Alino commented 6 years ago

closed with commit https://github.com/GALAglobal/TAPICC-API-implementation/commit/db905d532d7fdb9b5528920ad0b48596e9c321ee and swagger definition was changed accordingly