-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
receipt form api endpoints #128
Comments
[POST] /get-receipts/{userId}?limit=X&offset=Y
|
what about the following instead? GET /users/{userId}/receipts?limit=X&offset=Y |
Yeah that works, I like that. |
@Lev1ty What is the reason for switching to query string params instead of keeping everything in the request body? All of the other endpoints at this point have been part of the request body so I'm thinking we keep it consistent if possible. |
|
If we want to include a searching feature where we can filter by a search parameter and query, we could use POST and pass the search parameters in the body. For now, what @Lev1ty mentioned in 4 above is sufficient imo. |
Receipts (Expenses)
@Zack-Ren please add more endpoints if I missed any or new ones come up!
The text was updated successfully, but these errors were encountered: