Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove sub-request functionality #2044

Closed
codeguy opened this issue Nov 19, 2016 · 2 comments
Closed

Remove sub-request functionality #2044

codeguy opened this issue Nov 19, 2016 · 2 comments
Assignees
@codeguy
Labels
Slim 4

Comments

@codeguy
Copy link
Member

codeguy commented Nov 19, 2016

No description provided.

@codeguy codeguy added the Slim 4 label Nov 19, 2016
@codeguy codeguy added this to the 4.0 milestone Nov 19, 2016
@codeguy codeguy self-assigned this Nov 19, 2016
@far-blue
Copy link

out of curiosity, are sub-requests being superseded by something 'better' or just considered a bad idea?

I am currently investigating using sub-requests to support multiplexed update requests via PATCH - where the PATCH payload is actually a batch of partial updates to possibly multiple different records and which need to be unpacked and dispatched as if they were individual PUT / POST / DELETE requests. Kind of like https://tools.ietf.org/html/rfc6902 but for REST requests. This reduces the number of individual REST requests.

If not sub-requests, any suggestions on how this could still work?

codeguy added a commit to codeguy/Slim that referenced this issue Nov 26, 2016
@codeguy
Remove subrequest feature. Fixes slimphp#2044.
263ce35
akrabat added a commit that referenced this issue Nov 27, 2016
@akrabat
Merge branch 'codeguy-4.x-remove-subrequest' into 4.x
e94befb 
Closes #2078
Fixes #2044
@codeguy
Copy link
Member Author

codeguy commented Dec 4, 2016

Resolved.

@codeguy codeguy closed this as completed Dec 4, 2016
@akrabat akrabat removed this from the 4.0 milestone Sep 16, 2018
@l0gicgate l0gicgate mentioned this issue Feb 12, 2019
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@codeguy codeguy

Labels
Slim 4
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@codeguy @akrabat @far-blue