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

Update README to remove event priority note #57

Merged

michaeldyrynda merged 1 commit into michaeldyrynda:main from decaylala:patch-1

Nov 22, 2022

Contributor

decaylala commented Nov 10, 2022

The event priority had been removed in Laravel 5.4. The additional note in the README doesn't work now, so maybe it can be removed.

Event Priority

Support for event handler "priorities" has been removed. This undocumented feature typically indicates an abuse of the event feature. Instead, consider using a series of synchronous method calls. Alternatively, you may dispatch a new event from within the handler of another event in order to ensure that a given event's handler fires after an unrelated handler.


          Update README to remove event priority note

edd00b5

michaeldyrynda merged commit 6dc274b into michaeldyrynda:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment