-
Notifications
You must be signed in to change notification settings - Fork 295
Resolving: Error on Implementation in Laravel 5.3 #186
Conversation
…th Model::is(). Changes contains params L5.3 L5.3: contains($value, key) instead of contains($key, $value)
Compatibility with L5.3. Change is() method to isRole(). Conflicts with Model::is()
@romanbican Can you merge this pull request? This is an urgent one for me. |
Works perfect |
yea please merge! :) +1 |
Hey guys, To fix it temporary use this: II. Change version of the package in composer.json
Hopefully it may help. @romanbican can you please make respective changes from https://github.com/emiliomedina21/roles.git or pull that |
@thachp you can migrate to https://github.com/Zizaco/entrust in case it's very urgent for you. It took me about 30 mins to update my project to Entrust. Entrust and this one are very similar but Entrust seems to work well with Laravel 5.3. Will update you in case any issues will be found for Entrust too. |
Also created a similar package to this one resolving a couple of issues between then Laravel 5 and now 5.3. I however removed permissions in favor of Laravel's authorization. Plus one other minor change. For those that use roles alone it should be a seamless implementation. |
We decided to update and maintain a copy of bican roles and merged this pull request. |
Compatibility with L5.3. Change is() method to isRole(). Conflicts with Model::is()
@emiliomedina21
#185