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.

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

!!! TASK: Deprecate and replace ActionResponse in dispatcher #3294

Conversation

mhsdesign
Copy link
Member

@mhsdesign mhsdesign commented Jan 29, 2024

Resolves: #3289 (Contains discussion about the reasoning).

will not be merged directly into 9.0 but included in this mvc overhaul pr: #3311

(original pr in christians fork kitsunet#5)

class Dispatcher
{
    public function dispatch(ActionRequest $request): ResponseInterface;
}

@mhsdesign mhsdesign force-pushed the task/makeActionResponseObsolete branch from a400e42 to 2711954 Compare February 6, 2024 12:40
@mhsdesign mhsdesign force-pushed the task/makeActionResponseObsolete branch from 2711954 to 3b5d2bb Compare February 6, 2024 12:44
@mhsdesign mhsdesign force-pushed the task/makeActionResponseObsolete branch from 3b5d2bb to b4b1d23 Compare February 6, 2024 14:06
@mhsdesign mhsdesign changed the title TASK: Make ActionResponse obsolete TASK: Deprecate and replace ActionResponse in dispatcher Feb 6, 2024
@mhsdesign mhsdesign changed the title TASK: Deprecate and replace ActionResponse in dispatcher !!! TASK: Deprecate and replace ActionResponse in dispatcher Feb 6, 2024
@mhsdesign
Copy link
Member Author

mhsdesign commented Feb 22, 2024

TODO: Test with media module as it has pagination widget.

-> ✅

@mhsdesign mhsdesign force-pushed the task/makeActionResponseObsolete branch from 1ed9617 to ff2c7da Compare March 3, 2024 08:11
This will not open a memory stream
`replaceHttpResponse` is unsafe to use and has hard to tell behaviour.
Instead, we up-merge the headers explicitly
@mhsdesign mhsdesign force-pushed the task/makeActionResponseObsolete branch from ff2c7da to 3d32ace Compare March 3, 2024 08:14
@mhsdesign mhsdesign marked this pull request as ready for review March 3, 2024 08:31
@mhsdesign mhsdesign merged commit 415584b into neos:feature/dispatcher-returns-psr-responses Mar 3, 2024
2 checks passed
mhsdesign added a commit to kitsunet/neos-development-collection that referenced this pull request Mar 3, 2024
mhsdesign added a commit to mhsdesign/flow-development-collection that referenced this pull request Oct 27, 2024
neos-bot pushed a commit to neos/flow that referenced this pull request Oct 28, 2024
neos-bot pushed a commit to neos/flow that referenced this pull request Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant