REST Controller: Remove endpoint versions #120

New issue

Closed

opened 2018-02-19 00:23:35 +00:00 by jamie · 6 comments

jamie commented

2018-02-19 00:23:35 +00:00

(Migrated from git.hazaar.io)

This is too restrictive and without this feature it can still be implemented manually without any more effort than the feature provides.

Currently you would:

/**
 * @route('/test')
 * @version 1,2
 */
function myTest(){
  //Do stuff
}

This would provide the test endpoint at both /api/v1/test and /api/v2/test.

The problem is if you don't want to use versions, you can't. Everything has to be prefixed with a version which is very restrictive. Removing this would allow the above endpoint to exist at /api/test.

You will still be able to implement the original functionality quite simply by doing this:

/**
 * @route('/v1/test')
 * @route('/v2/test')
 */
function myTest(){
  //Do stuff
}

This is too restrictive and without this feature it can still be implemented manually without any more effort than the feature provides. Currently you would: ```php /** * @route('/test') * @version 1,2 */ function myTest(){ //Do stuff } ``` This would provide the ```test``` endpoint at both ```/api/v1/test``` and ```/api/v2/test```. The problem is if you don't want to use versions, you can't. Everything has to be prefixed with a version which is very restrictive. Removing this would allow the above endpoint to exist at ```/api/test```. You will still be able to implement the original functionality quite simply by doing this: ```php /** * @route('/v1/test') * @route('/v2/test') */ function myTest(){ //Do stuff } ```

jamie commented

2018-02-21 04:29:46 +00:00

(Migrated from git.hazaar.io)

created branch 120-rest-controller-remove-endpoint-versions

created branch [`120-rest-controller-remove-endpoint-versions`](https://git.hazaarlabs.com/hazaar/hazaar-mvc/compare/master...120-rest-controller-remove-endpoint-versions)

jamie commented

2018-02-21 04:30:32 +00:00

(Migrated from git.hazaar.io)

mentioned in merge request !35