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

Fix test broken by Laravel 9.52.6 / 10.8.0 #237

Closed
wants to merge 1 commit into from
Closed

Fix test broken by Laravel 9.52.6 / 10.8.0 #237

wants to merge 1 commit into from

Conversation

dmason30
Copy link
Contributor

@dmason30 dmason30 commented May 7, 2023

Caused by laravel/framework#46806

The afterResponse handler was changed from using $this->dispatchNow(...) to $this->dispatchSync(...) in the above PR.

@lorisleiva
Copy link
Owner

Thanks for that, this pushed me in the right direction. There were more issues but I managed to fix them in a separate PR so I'll close this one. 🍺

@lorisleiva lorisleiva closed this May 8, 2023
@dmason30 dmason30 deleted the test-fix-laravel-9.52.6 branch May 8, 2023 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants