search

crusttech / crust-server

Apache License 2.0
87 stars 21 forks source link

fix(messaging): removed double permission word in path #41

Closed mitjaziv closed 5 years ago

mitjaziv commented 5 years ago

Shorten permission path:

/permissions/* - with 10 handlers, 3 subroutes
    GET /permissions/ -> github.com/crusttech/crust/messaging/rest/handlers.NewPermissions.func1
    GET /permissions/scopes/{scope} -> github.com/crusttech/crust/messaging/rest/handlers.NewPermissions.func4
    POST    /permissions/{teamID} -> github.com/crusttech/crust/messaging/rest/handlers.NewPermissions.func3
    GET /permissions/{teamID} -> github.com/crusttech/crust/messaging/rest/handlers.NewPermissions.func2

instead of:

/permissions/* - with 10 handlers, 3 subroutes
    GET /permissions/permissions -> github.com/crusttech/crust/messaging/rest/handlers.NewPermissions.func1
    GET /permissions/permissions/scopes/{scope} -> github.com/crusttech/crust/messaging/rest/handlers.NewPermissions.func4
    GET /permissions/permissions/{teamID} -> github.com/crusttech/crust/messaging/rest/handlers.NewPermissions.func2
    POST    /permissions/permissions/{teamID} -> github.com/crusttech/crust/messaging/rest/handlers.NewPermissions.func3