-
Notifications
You must be signed in to change notification settings - Fork 151
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
Support for Laravel 5.8 #129
base: master
Are you sure you want to change the base?
Conversation
The fire method (which was deprecated in Laravel 5.4) of the Illuminate/Events/Dispatcher class has been removed. You should use the dispatch method instead.
add php 7.2 env
I can confirm this PR fixes the event fire error. |
I got the same issue while trying to upgrade to Laravel 5.8 and the PR fix it. Will it be available soon? |
Approved but not merged....? |
Any movement on this @hootlex? Thanks! |
Some news about the pull request? |
@hootlex hoping for a merge for Laravel 5.8 support! |
Is the repo still maintained? Did someone else take over? |
@kainxspirits - might as well update this PR to add Laravel 6.0 support as well ;) |
@hootlex Please fill in these edits to support new versions of laravel |
Already made a PR on MoeCasts fork, but it might be quicker to just make a new PR here.
You can delete this PR if another fix is merged.
Next version should probably be 1.1, since we drop support for previous php version.