search

GALAglobal / TAPICC-API-implementation

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

Nicer smaller routes #48

Closed Alino closed 5 years ago

Alino commented 6 years ago

What if we change the routes to make them smaller and nicer?

from

POST /jobs/{jobId}/assets/uploadfile
GET /jobs/{jobId}/assets/{assetId}/downloadfile
GET /jobs/{jobId}/assets
GET, PUT, DELETE /jobs/{jobId}/assets/{assetId}

to

POST /assets/uploadfile
GET /assets/{assetId}/downloadfile
GET /assets?jobId={jobId}
GET, PUT, DELETE /assets/{assetId}
Alino commented 6 years ago

this is consistent with what we did with Tasks already here https://github.com/GALAglobal/TAPICC-API-implementation/issues/47

Alino commented 5 years ago

I am planning to implement this next week Friday (giving you time to disagree), with all routes. There will be no nested routes anymore.

Alino commented 5 years ago

changed in swagger https://github.com/GALAglobal/TAPICC-API-implementation/commit/0e61cc44da450119cbe2746a3bcc20d0ace1295b