-
Notifications
You must be signed in to change notification settings - Fork 43
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
Fixed all users online list #29
Conversation
request to master
* Correcting flow of middlewares
Configuring Codacy and adding badges.
- Changed method "boot" on UsersOnlineEventServiceProvider to comply …
Table of versions
* New method to return all users online * README updated
* Compatible with Laravel 5.4
Fixture/5.4 clean
Update fork Develop
* Improving changelog
@luancschmitz what package version and laravel version that are you using? |
Hi. I'm using version 5.3 of Laravel and version 2.0 of the package.
|
agora vi que você é brasileiro, Eu só modifiquei o código listado. Não conheço muito desse pull request, talvez eu tenha feito errada submissão do erro e a solução. |
If you are using an version less then 5.4 on Laravel, the method filter don't work, please read the README https://github.com/highideas/laravel-users-online/tree/2.0.0 to see how works on this version: |
Upgrade to Laravel 5.5 on master
Broken links
I encountered a problem when using the allOnline method. He said the $filter method was not recognized. For this I changed the allOnline method of:
for this: