jaxtell - added get payment by user id functionality#28
jaxtell - added get payment by user id functionality#28jaxtell wants to merge 1 commit intoBravoLT:mainfrom
Conversation
|
|
||
| import javax.servlet.http.HttpServletResponse; | ||
|
|
||
| @RequestMapping(value = "/payment") |
There was a problem hiding this comment.
Any comment on how the endpoint should look if you were to design it without pattern following?
There was a problem hiding this comment.
It depends on how it would be used. If user id is always going to be present it might make sense to be /user/{userId}/payment. If there are future requests that allow searching across users then I'd leave it top level. Instead of using words in the URL I'd favor http verbs to indicate action, e.g. GET on /payment/user/{userId} to get all payments for a user and/or GET on /payment with parameters to get payment(s) based on query parameters, PUT on /payment to add a payment, PATCH on /payment to update a payment, POSTs on /payment/[ACTION] for more complex operations. I'd want to balance keeping the URL short while still being descriptive of what it does.
2 participants
No description provided.